Skip to content

Feat: Allow forms to use their own magic-link config#383

Merged
emilyjevans merged 3 commits into
ukhsa-collaboration:v2from
SullyK:kls-magic-link
May 28, 2025
Merged

Feat: Allow forms to use their own magic-link config#383
emilyjevans merged 3 commits into
ukhsa-collaboration:v2from
SullyK:kls-magic-link

Conversation

@SullyK

@SullyK SullyK commented May 28, 2025

Copy link
Copy Markdown
  • Previously, all users were redirected to CareOBRA's form via a hardcoded config in MagicLinkRedirectController.ts

  • Updated the controller to allow each form to specify its own magicLinkConfig, enabling flexible redirects per form

  • Fallback remains for undefined, mainly for backwards compat purposes

Example to add to main form:

"magicLinkConfig": "kls-magic-link",

replace kls-magic-link with your own magic-link config.

  • [✓] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Locally.

Checklist:

  • [✓] I have performed a self-review of my own code
  • [✓] I have commented my code, particularly in hard-to-understand areas

SullyK and others added 2 commits May 28, 2025 07:17
- Previously, all users were redirected to CareOBRA's form via a hardcoded config
  in MagicLinkRedirectController.ts

- Updated the controller to allow each form to specify its own magicLinkConfig,
  enabling flexible redirects per form

- Fallback remains for undefined, mainly for backwards compat purposes
@SullyK SullyK marked this pull request as ready for review May 28, 2025 06:25
@SullyK SullyK requested a review from a team as a code owner May 28, 2025 06:25
kathryn-dale
kathryn-dale previously approved these changes May 28, 2025
@SullyK

SullyK commented May 28, 2025

Copy link
Copy Markdown
Author

ready to be merged.

@emilyjevans emilyjevans merged commit a93d551 into ukhsa-collaboration:v2 May 28, 2025
1 check failed
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.

3 participants