Skip to content

Update (Form): Updated schema and email template to match changes for redirect-individual-mail#82

Merged
phrogwrld merged 2 commits intodevelopfrom
chore/redirect-personal-mail-changes
Jan 29, 2026
Merged

Update (Form): Updated schema and email template to match changes for redirect-individual-mail#82
phrogwrld merged 2 commits intodevelopfrom
chore/redirect-personal-mail-changes

Conversation

@IsaiahSama
Copy link
Contributor

@IsaiahSama IsaiahSama commented Jan 29, 2026

Description

Updated validation schema on the backend to match the content updates.
Updated email template to show information added (adult dependents)

Type of Change

  • Bug fix
  • New feature
  • Breaking change: Update payloads
  • Documentation update

Changes Made

  • schemas/post-office-redirection-individual.json
  • src/email/templates/post-office-redirection-notice.hbs

Notes

Testing

  • Manual tests completed
  • Added unit tests
  • Added e2e tests

Related Github Issue(s)/Trello Ticket(s)

Re: 403

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Documentation updated

@amazon-inspector-n-virginia
Copy link

⏳ I'm reviewing this pull request for security vulnerabilities and code quality issues. I'll provide an update when I'm done

@IsaiahSama IsaiahSama marked this pull request as draft January 29, 2026 20:27
@amazon-inspector-n-virginia
Copy link

✅ I finished the code review, and didn't find any security or code quality issues.

@IsaiahSama IsaiahSama marked this pull request as ready for review January 29, 2026 20:41
@IsaiahSama IsaiahSama requested a review from phrogwrld January 29, 2026 21:55
Copy link
Contributor

@phrogwrld phrogwrld left a comment

Choose a reason for hiding this comment

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

LGTM!

@phrogwrld phrogwrld merged commit ba84e79 into develop Jan 29, 2026
2 checks passed
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