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

Improve decline email and view content #1796

Merged
merged 3 commits into from
Nov 14, 2023
Merged

Improve decline email and view content #1796

merged 3 commits into from
Nov 14, 2023

Conversation

thomasleese
Copy link
Contributor

@thomasleese thomasleese commented Nov 9, 2023

This updates the content on the decline email and view as requested by our content designers. I've also refactored the view object to make it easy to use in the mailer and the view.

Trello Card

Screenshots

Screenshot 2023-11-09 at 17 11 35

@thomasleese thomasleese marked this pull request as ready for review November 9, 2023 16:45
@thomasleese thomasleese requested a review from a team as a code owner November 9, 2023 16:45
Copy link

github-actions bot commented Nov 9, 2023

Copy link
Contributor

@syed87 syed87 left a comment

Choose a reason for hiding this comment

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

looks good to me

This changes the content we show for the reference number in various
views as requested by our content designers.
I've renamed it and combined the decline reason methods together
so we can use them in the views and in the mailers.
This updates the content of the decline email and view to match the
changes requested by our content designers.
@thomasleese thomasleese merged commit f65b24a into main Nov 14, 2023
12 checks passed
@thomasleese thomasleese deleted the decline-content branch November 14, 2023 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants