Skip to content

Site Wide Banner #2403

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

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Conversation

A-Wheeto
Copy link
Contributor

@A-Wheeto A-Wheeto commented Apr 14, 2025

Status

Review progress:

  • Browser tested
  • Front-end review completed
  • Tech review completed

What's changed?

  • Implemented site wide banner component driven by Strapi.
  • Banner runs via the application controller and will show the latest banner if the date/time is current

Steps to perform after deploying to production

If the production environment requires any extra work after this PR has been deployed detail it here. This could be running a Rake task, migrating a DB table, or upgrading a Gem. That kind of thing.

@A-Wheeto A-Wheeto force-pushed the 2922-introduce-a-sitewide-banner-feature branch from 8693289 to 4f9c130 Compare April 15, 2025 12:31
@tc-deploybot tc-deploybot temporarily deployed to teachcomputing-pr-2403 April 23, 2025 07:16 Inactive
@A-Wheeto A-Wheeto temporarily deployed to teachcomputing-pr-2403 April 23, 2025 14:32 Inactive
@A-Wheeto A-Wheeto temporarily deployed to teachcomputing-pr-2403 April 24, 2025 07:01 Inactive
A-Wheeto added 7 commits May 13, 2025 14:11
New single cms resource
New cms view component
…er to enter start time and end time for the banner

Created site wide banner stub - added into spec helper
Logic to check for current banners and display the latest one
Added date query into graphql
Changed text to rich header to allow for links to be placed into the banners
Added some testing on header and view component
@A-Wheeto A-Wheeto force-pushed the 2922-introduce-a-sitewide-banner-feature branch from dbf30b6 to dc9592b Compare May 13, 2025 14:34
@tc-deploybot tc-deploybot temporarily deployed to teachcomputing-pr-2403 May 13, 2025 15:08 Inactive
@A-Wheeto A-Wheeto temporarily deployed to teachcomputing-pr-2403 May 14, 2025 12:58 Inactive
@A-Wheeto A-Wheeto temporarily deployed to teachcomputing-pr-2403 May 14, 2025 13:12 Inactive
Copy link

@tc-deploybot tc-deploybot temporarily deployed to teachcomputing-pr-2403 May 30, 2025 07:41 Inactive
@markjs
Copy link

markjs commented Jun 19, 2025

Looks good, straightforward stuff 👍

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.

3 participants