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

Closed
wants to merge 45 commits into from
Closed

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.

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

@urumo urumo closed this Nov 5, 2024
@argon-chat argon-chat locked and limited conversation to collaborators Nov 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants