Skip to content

Only show CSA Before Workshop note for Summer Workshop subject - #51353

Merged
TurnerRiley merged 3 commits into
stagingfrom
only-show-csa-before-workshop-note-in-summer-ws
Apr 17, 2023
Merged

Only show CSA Before Workshop note for Summer Workshop subject#51353
TurnerRiley merged 3 commits into
stagingfrom
only-show-csa-before-workshop-note-in-summer-ws

Conversation

@TurnerRiley

@TurnerRiley TurnerRiley commented Apr 14, 2023

Copy link
Copy Markdown
Contributor

This PR refactors how we show the "Before the Workshop" section in workshop enrollment receipt and reminder emails to tailor the information better to the specific workshop case. This change affects both the enrollment receipt and enrollment reminder emails but it does not depend on whether a given email is a receipt or reminder email so I arbitrarily chose the receipt emails to demo below.

Case: CSA summer workshop email (no change expected)

Original:
old_csa_summer_workshop

New:
new_csa_summer_workshop

Case: CSA not summer workshop (e.g. CSA capstone in this case) email

Original:
old_csa_non_summer_workshop

New:
new_csa_non_summer_workshop

Case: Not CSA (e.g. CSD in this case) summer workshop email (no change expected)

Original:
old_non_csa_summer_workshop

New:
new_non_csa_summer_workshop

Links

Jira ticket: here

Testing story

Local testing.

PR Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

@TurnerRiley
TurnerRiley requested a review from a team April 14, 2023 17:14
@TurnerRiley
TurnerRiley merged commit 476fab1 into staging Apr 17, 2023
@TurnerRiley
TurnerRiley deleted the only-show-csa-before-workshop-note-in-summer-ws branch April 17, 2023 16:15
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