Remove Plus users' manual removals' special-casing #4534
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
References:
Jira: MNTOR-3172
Figma: https://www.figma.com/file/CaEKIhvSJqf6KNIMzSkt40/Concepts-for-Monitor-MVP-Redesign?type=design&node-id=11165-1860&mode=design&t=s1UG53aIWQeHAz2A-4
Description
Apparently we'll be sending them the same monthly activity emails
as Plus users who have not manually resolved anything.
Note that I also fixed a bug where I forgot to include the generic email styles with other emails than the monthly activity email with manual resolutions. (Ironically, I called out that I was afraid for forgetting that in the PR where I added the styles - only to do so in the very PR I called that out. "I suppose in practice people will start by copy-pasting an existing template" - except I didn't update that template 馃檭 )
How to test
I left in the story with manual resolutions in Storybook, so you can compare that with the one without and see that they're the same.
Checklist (Definition of Done)