Why?
What is the problem this PR attempts to solve, and why is it important? Fixing a bug? Explain the problem you're solving. Laying a foundation for future work? Explain how this fits into the bigger picture. Don't forget to link to the Clubhouse issue above.
What?
What does this PR change, and how does that solve the problem noted above? Call out any related changes, and add before-and-after screenshots for UI updates.
Caveats
Are there downsides or side-effects that should be weighed against this update? Any lingering unknowns or things you can’t test without production data or traffic?
Testing Notes
Is any special setup required to test this change? Non-obvious things that should be checked?
A list of things to test:
- [ ] Test item 1
- [ ] Test item 2
- [ ] Test item 3
Deploy Tasks
Note anything that needs to be done before, during, or after this change is deployed. Also add the “deploy task” label to this PR and the Clubhouse issue so it's obvious the next time we make a deployment.
- [ ] Update the database things
- [ ] Tell Client Experience this issue is now resolved
- [ ] Confirm that emails are still being sent
Alternatives Considered
Were there other approaches or solutions to this problem which you considered? Why were they not chosen?
Further Reading
Were there articles or StackOverflow answers you found especially eye-opening when working on this? Slack conversation around this? Provide a link to the thread.
Merge Instructions
Please DO NOT squash my commits when merging