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

[2027] Update apply deadline banner in publish #4360

Merged
merged 3 commits into from
Jul 10, 2024

Conversation

elceebee
Copy link
Contributor

Context

We’ve identified some banners in publish that need updating as part of the end of cycle.

This ticket is just related to the banner users see after 30 July, alerting them of the apply deadline.

We agreed in this slack thread that we would use the same content as the banner we show candidates who are applying for courses

https://trello.com/c/myMbTZ4Q

Changes proposed in this pull request

  • Updated text
  • Added preview for deadline banner

Before
image

After
image

Guidance to review

Checklist

  • Make sure all information from the Trello card is in here
  • Attach to Trello card
  • Rebased main
  • Cleaned commit history
  • Tested by running locally
  • Inform data insights team due to database changes

@elceebee elceebee marked this pull request as ready for review July 10, 2024 13:18
@elceebee elceebee requested a review from avinhurry July 10, 2024 13:24
@elceebee elceebee changed the title 2027 update apply deadline banner in publish [2027] Update apply deadline banner in publish Jul 10, 2024
@elceebee elceebee requested a review from dcyoung-dev July 10, 2024 13:25
@elceebee
Copy link
Contributor Author

@elceebee elceebee force-pushed the 2027-update-apply-deadline-banner-in-publish branch from 9bab674 to 1131f05 Compare July 10, 2024 13:31
@elceebee elceebee merged commit 22c8b79 into main Jul 10, 2024
19 checks passed
@elceebee elceebee deleted the 2027-update-apply-deadline-banner-in-publish branch July 10, 2024 14:09
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