Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User identity #2

Draft
wants to merge 45 commits into
base: master
Choose a base branch
from
Draft

User identity #2

wants to merge 45 commits into from

Conversation

urumo
Copy link
Contributor

@urumo urumo commented Oct 15, 2024

No description provided.

@urumo urumo requested a review from 0xF6 October 15, 2024 15:08
@urumo urumo self-assigned this Oct 15, 2024
@urumo urumo marked this pull request as draft October 15, 2024 15:10
</PropertyGroup>

<ItemGroup>
<PackageReference Include="Microsoft.AspNetCore.Identity.UI" Version="8.0.10"/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.UI
Зачем?


namespace Argon.Api.Common.Models;

public class ApplicationUser : IdentityUser<Guid>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AspNetCore.Identity не научился в рекорды?

Comment on lines +5 to +7
public class ApplicationUser : IdentityUser<Guid>
{
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
public class ApplicationUser : IdentityUser<Guid>
{
}
public class ApplicationUser : IdentityUser<Guid>;

{
public Task SendConfirmationLinkAsync(ApplicationUser user, string email, string confirmationLink)
{
logger.LogInformation(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

реализация

options.ClusterId = nameof(Api);
options.ServiceId = nameof(Api);
});
siloBuilder.ConfigureEndpoints(11111, 30000, listenOnAnyHostAddress: true);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

я бы все таки в конфигурацию вынес эти порты

builder.Services.AddCors(options =>
{
options.AddDefaultPolicy(
policy => { policy.AllowAnyHeader().AllowAnyMethod().AllowAnyOrigin(); });
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

в конфигурацию

@@ -0,0 +1,5 @@
namespace Argon.Grains;

public class Class1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

зачем это?

@@ -0,0 +1,5 @@
namespace Argon.Orleans.Client;

public class Class1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тоже самое

@@ -0,0 +1,18 @@
<Project Sdk="Microsoft.NET.Sdk.BlazorWebAssembly">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

зачем тут клиент васма

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants