Skip to content

Updates to PD registration flow #10777

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

Merged
merged 2 commits into from
May 16, 2025
Merged

Updates to PD registration flow #10777

merged 2 commits into from
May 16, 2025

Conversation

jimchamp
Copy link
Collaborator

@jimchamp jimchamp commented May 14, 2025

Makes the following changes to the PD flow when a patron first logs in:

  • unqualifed pda values are captured on the account
  • pda and rpd values are written to the account before the email is sent
  • Adds a new rpd status code (this may require discussion), and encodes these as an Enum

New login flow:

  1. Patron with pda cookie logs in.
  2. The pda value is added to the patron's account
  3. rpd is added to the account, with a value of 0 (signifying that they are requesting special access).
  4. "Next steps" email is sent to patron.
  5. Account's rpd value is updated to 1 (meaning that the email has been sent).
  6. pda cookie is unset and expired.

Technical

Testing

Screenshot

Stakeholders

@jimchamp jimchamp marked this pull request as draft May 14, 2025 20:08
@jimchamp jimchamp changed the title Capture unqualified pda values Updates to PD registration flow May 16, 2025
@jimchamp jimchamp marked this pull request as ready for review May 16, 2025 21:48
@mekarpeles mekarpeles merged commit 959e1ac into master May 16, 2025
8 checks passed
@mekarpeles mekarpeles deleted the pd-reg-flow-updates branch May 16, 2025 22:21
RayBB pushed a commit that referenced this pull request May 21, 2025
* Capture `unqualified` `pda` values
* Set `pda` on account before sending email
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.

2 participants