Fix registration bugs, update input design, and improve validation logic

application/account-management/Api/AccountRegistrations/AccountRegistrationsEndpoints.cs

@@ -17,9 +17,9 @@ public void MapEndpoints(IEndpointRouteBuilder routes)
             => await mediator.Send(query)
         ).Produces<bool>();
 
-        group.MapPost("/start", async Task<ApiResult> (StartAccountRegistrationCommand command, ISender mediator)
-            => (await mediator.Send(command)).AddResourceUri(RoutesPrefix)
-        );
+        group.MapPost("/start", async Task<ApiResult<StartAccountRegistrationResponse>> (StartAccountRegistrationCommand command, ISender mediator)
+            => await mediator.Send(command)
+        ).Produces<StartAccountRegistrationResponse>();
 
         group.MapPost("{id}/complete", async Task<ApiResult> (AccountRegistrationId id, CompleteAccountRegistrationCommand command, ISender mediator)
             => await mediator.Send(command with { Id = id })

application/account-management/Application/AccountRegistrations/StartAccountRegistration.cs

@@ -12,14 +12,16 @@
 namespace PlatformPlatform.AccountManagement.Application.AccountRegistrations;
 
 public sealed record StartAccountRegistrationCommand(string Subdomain, string Email)
-    : ICommand, IRequest<Result<AccountRegistrationId>>
+    : ICommand, IRequest<Result<StartAccountRegistrationResponse>>
 {
     public TenantId GetTenantId()
     {
         return new TenantId(Subdomain);
     }
 }
 
+public sealed record StartAccountRegistrationResponse(string AccountRegistrationId, int ValidForSeconds);
+
 public sealed class StartAccountRegistrationValidator : AbstractValidator<StartAccountRegistrationCommand>
 {
     public StartAccountRegistrationValidator(ITenantRepository tenantRepository)
@@ -40,23 +42,23 @@
     IEmailService emailService,
     IPasswordHasher<object> passwordHasher,
     ITelemetryEventsCollector events
-) : IRequestHandler<StartAccountRegistrationCommand, Result<AccountRegistrationId>>
+) : IRequestHandler<StartAccountRegistrationCommand, Result<StartAccountRegistrationResponse>>
 {
-    public async Task<Result<AccountRegistrationId>> Handle(StartAccountRegistrationCommand command, CancellationToken cancellationToken)
+    public async Task<Result<StartAccountRegistrationResponse>> Handle(StartAccountRegistrationCommand command, CancellationToken cancellationToken)
     {
         var existingAccountRegistrations
             = accountRegistrationRepository.GetByEmailOrTenantId(command.GetTenantId(), command.Email);
 
         if (existingAccountRegistrations.Any(r => !r.HasExpired()))
         {
-            return Result<AccountRegistrationId>.Conflict(
+            return Result<StartAccountRegistrationResponse>.Conflict(
                 "Account registration for this subdomain/mail has already been started. Please check your spam folder."
             );
         }
 
         if (existingAccountRegistrations.Count(r => r.CreatedAt > TimeProvider.System.GetUtcNow().AddDays(-1)) > 3)
         {
-            return Result<AccountRegistrationId>.TooManyRequests("Too many attempts to register this email address. Please try again later.");
+            return Result<StartAccountRegistrationResponse>.TooManyRequests("Too many attempts to register this email address. Please try again later.");
         }
 
         var oneTimePassword = GenerateOneTimePassword(6);
@@ -75,7 +77,7 @@
             cancellationToken
         );
 
-        return accountRegistration.Id;
+        return new StartAccountRegistrationResponse(accountRegistration.Id, accountRegistration.GetValidForSeconds());
     }
 
     public static string GenerateOneTimePassword(int length)

application/account-management/Domain/AccountRegistrations/AccountRegistration.cs

@@ -7,14 +7,15 @@ namespace PlatformPlatform.AccountManagement.Domain.AccountRegistrations;
 public sealed class AccountRegistration : AggregateRoot<AccountRegistrationId>
 {
     public const int MaxAttempts = 3;
+    private const int ValidForSeconds = 300;
 
     private AccountRegistration(TenantId tenantId, string email, string oneTimePasswordHash)
         : base(AccountRegistrationId.NewId())
     {
         TenantId = tenantId;
         Email = email;
         OneTimePasswordHash = oneTimePasswordHash;
-        ValidUntil = CreatedAt.AddMinutes(5);
+        ValidUntil = CreatedAt.AddSeconds(ValidForSeconds);
     }
 
     public TenantId TenantId { get; private set; }
@@ -56,6 +57,11 @@ public void MarkAsCompleted()
 
         Completed = true;
     }
+
+    public int GetValidForSeconds()
+    {
+        return Convert.ToInt16((ValidUntil - TimeProvider.System.GetUtcNow()).TotalSeconds);
+    }
 }
 
 [TypeConverter(typeof(StronglyTypedIdTypeConverter<string, AccountRegistrationId>))]

application/account-management/WebApp/pages/(auth)/login.tsx

@@ -0,0 +1,17 @@
+import { createFileRoute } from "@tanstack/react-router";
+import { HorizontalHeroLayout } from "@/shared/ui/layout/HorizontalHeroLayout";
+import { LoginForm } from "@/shared/ui/auth/LoginForm";
+import { ErrorMessage } from "@/shared/ui/auth/ErrorMessage";
+
+export const Route = createFileRoute("/(auth)/login")({
+  component: () => (
+    <HorizontalHeroLayout>
+      <LoginForm />
+    </HorizontalHeroLayout>
+  ),
+  errorComponent: (props) => (
+    <HorizontalHeroLayout>
+      <ErrorMessage {...props} />
+    </HorizontalHeroLayout>
+  )
+});

application/account-management/WebApp/pages/(auth)/register/expired.tsx

@@ -0,0 +1,17 @@
+import { createFileRoute } from "@tanstack/react-router";
+import { HorizontalHeroLayout } from "@/shared/ui/layout/HorizontalHeroLayout";
+import { ErrorMessage } from "@/shared/ui/auth/ErrorMessage";
+import { VerificationCodeExpiredMessage } from "@/shared/ui/auth/VerificationCodeExpiredMessage";
+
+export const Route = createFileRoute("/(auth)/register/expired")({
+  component: () => (
+    <HorizontalHeroLayout>
+      <VerificationCodeExpiredMessage />
+    </HorizontalHeroLayout>
+  ),
+  errorComponent: (props) => (
+    <HorizontalHeroLayout>
+      <ErrorMessage {...props} />
+    </HorizontalHeroLayout>
+  )
+});

application/account-management/WebApp/pages/(auth)/register/index.tsx

@@ -0,0 +1,17 @@
+import { createFileRoute } from "@tanstack/react-router";
+import { HorizontalHeroLayout } from "@/shared/ui/layout/HorizontalHeroLayout";
+import { StartAccountRegistrationForm } from "@/shared/ui/auth/StartAccountRegistrationForm";
+import { ErrorMessage } from "@/shared/ui/auth/ErrorMessage";
+
+export const Route = createFileRoute("/(auth)/register/")({
+  component: () => (
+    <HorizontalHeroLayout>
+      <StartAccountRegistrationForm />
+    </HorizontalHeroLayout>
+  ),
+  errorComponent: (props) => (
+    <HorizontalHeroLayout>
+      <ErrorMessage {...props} />
+    </HorizontalHeroLayout>
+  )
+});

application/account-management/WebApp/pages/(auth)/register/verify.tsx

@@ -0,0 +1,17 @@
+import { createFileRoute } from "@tanstack/react-router";
+import { HorizontalHeroLayout } from "@/shared/ui/layout/HorizontalHeroLayout";
+import { CompleteAccountRegistrationForm } from "@/shared/ui/auth/CompleteAccountRegistrationForm";
+import { ErrorMessage } from "@/shared/ui/auth/ErrorMessage";
+
+export const Route = createFileRoute("/(auth)/register/verify")({
+  component: () => (
+    <HorizontalHeroLayout>
+      <CompleteAccountRegistrationForm />
+    </HorizontalHeroLayout>
+  ),
+  errorComponent: (props) => (
+    <HorizontalHeroLayout>
+      <ErrorMessage {...props} />
+    </HorizontalHeroLayout>
+  )
+});

application/account-management/WebApp/pages/index.tsx

@@ -1,22 +1,22 @@
 import { createFileRoute } from "@tanstack/react-router";
-import { CtaSection } from "./-landing/sections/CtaSection";
-import { FeatureSection } from "./-landing/sections/FeatureSection";
-import { FeatureSection2 } from "./-landing/sections/FeatureSection2";
-import { HeroSection } from "./-landing/sections/HeroSection";
-import { FeatureSection3 } from "./-landing/sections/FeatureSection3";
-import { TechnologySection } from "./-landing/sections/TechnologySection";
-import { TechnologySection2 } from "./-landing/sections/TechnologySection2";
-import { CommunitySection } from "./-landing/sections/CommunitySection";
-import { FeatureSection4 } from "./-landing/sections/FeatureSection4";
-import { CtaSection2 } from "./-landing/sections/CtaSection2";
-import { CtaSection3 } from "./-landing/sections/CtaSection3";
-import { FooterSection } from "./-landing/sections/FooterSection";
+import { CtaSection } from "../shared/ui/landingPage/sections/CtaSection";
+import { FeatureSection } from "../shared/ui/landingPage/sections/FeatureSection";
+import { FeatureSection2 } from "../shared/ui/landingPage/sections/FeatureSection2";
+import { HeroSection } from "../shared/ui/landingPage/sections/HeroSection";
+import { FeatureSection3 } from "../shared/ui/landingPage/sections/FeatureSection3";
+import { TechnologySection } from "../shared/ui/landingPage/sections/TechnologySection";
+import { TechnologySection2 } from "../shared/ui/landingPage/sections/TechnologySection2";
+import { CommunitySection } from "../shared/ui/landingPage/sections/CommunitySection";
+import { FeatureSection4 } from "../shared/ui/landingPage/sections/FeatureSection4";
+import { CtaSection2 } from "../shared/ui/landingPage/sections/CtaSection2";
+import { CtaSection3 } from "../shared/ui/landingPage/sections/CtaSection3";
+import { FooterSection } from "../shared/ui/landingPage/sections/FooterSection";
 
 export const Route = createFileRoute("/")({
-  component: Home
+  component: LandingPage
 });
 
-function Home() {
+function LandingPage() {
   return (
     <main className="flex w-full flex-col">
       <HeroSection />