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

LGA-3319 Add receive user answer route #1279

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

BenMillar-MOJ
Copy link
Contributor

@BenMillar-MOJ BenMillar-MOJ commented Oct 28, 2024

What does this pull request do?

Adds the receive answer route

This accepts a signed JWT payload containing:

  • The user's answers to the questions on Access Civil Legal Aid
  • The user's category of law, as determined by Access Civil Legal Aid
  • A harm flag if the user has indicated they or their children are at risk of harm
  • The destination the user should be redirected to, as determined by Access Civil Legal Aid

This route takes this payload, modifies the user's session if required, and redirects the the user to the appropriate next page.

Any other changes that would benefit highlighting?

Intentionally left blank.

Checklist

  • Provided JIRA ticket number in the title, e.g. "LGA-152: Sample title"

@BenMillar-MOJ BenMillar-MOJ requested a review from a team as a code owner October 28, 2024 19:41
Pheonnexx
Pheonnexx previously approved these changes Oct 30, 2024
TawneeOwl
TawneeOwl previously approved these changes Nov 8, 2024
Copy link
Contributor

@TawneeOwl TawneeOwl left a comment

Choose a reason for hiding this comment

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

LGTM

said-moj
said-moj previously approved these changes Nov 11, 2024
@BenMillar-MOJ BenMillar-MOJ dismissed stale reviews from said-moj and TawneeOwl via 34a6a23 November 11, 2024 16:53
@BenMillar-MOJ
Copy link
Contributor Author

@said-moj @TawneeOwl
Your reviews were automatically dismissed due to fixing a merge conflict: 34a6a23

Please can you re-review.

Copy link
Contributor

@TawneeOwl TawneeOwl left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

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.

4 participants