Simplifies text for email update#6655
Conversation
compwron
left a comment
There was a problem hiding this comment.
No tests are updated in this changeset - seems a bit strange. Are we missing tests? If so, are you interested in adding some in this PR or a later PR?
|
@compwron agreed! I can add or update the tests, sorry first PR and was just playing around with the repo. I also removed the Just wanted to point that out as I've never contributed to the project, but came across it on reddit!!! |
woops ---- I was wrong! @compwron do you want me to close and re-open PR when everything is passing or just keep it open? |
unconfirmed_email
|
just keep it open is fine :) |
|
sweet let me know what you think! |

What github issue is this PR for, if any?
Resolves #6636
What changed, and why?
It's an email, so the recipient doesn't need to see who it's addressed to - Removed recipient from email body
Updates text to read: Your CASA account's email has been updated to
How is this tested? (please write rspec and jest tests!) 💖💪
RSPEC + Jest Tests
Screenshots please :)
Run your local server and take a screenshot of your work! Try to include the URL of the page as well as the contents of the page.
Feelings gif (optional)
