Skip to content

Conversation

@BenjaminMichaelis
Copy link
Member

@BenjaminMichaelis BenjaminMichaelis commented Jul 10, 2025

Fixes #742

@BenjaminMichaelis BenjaminMichaelis self-assigned this Jul 10, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds a referral‐ID management step to the external login flow by injecting a referral service and calling it when a user signs in via an external provider.

  • Introduce IReferralService and configure it in the page model constructor.
  • In OnGetCallbackAsync, retrieve the user by external login and invoke EnsureReferralIdAsync.
Comments suppressed due to low confidence (1)

EssentialCSharp.Web/Areas/Identity/Pages/Account/ExternalLogin.cshtml.cs:24

  • The injected referralService parameter is never stored in a field, so calls to referralService in OnGetCallbackAsync will not compile. Add a private readonly IReferralService _referralService field and assign it in the constructor (e.g., _referralService = referralService).
    IReferralService referralService) : PageModel

@BenjaminMichaelis BenjaminMichaelis merged commit f51f159 into main Jul 10, 2025
5 checks passed
@BenjaminMichaelis BenjaminMichaelis deleted the fixReferralExternal branch July 10, 2025 16:20
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.

Hyperlinks are missing referrer id

2 participants