-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add release notes banner #1
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
Conversation
dc8a059 to
c8776fe
Compare
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds a release notes banner feature to the course authoring interface. The banner promotes the new release notes functionality with a dismissible notification that includes a link to the dedicated release notes page.
- Adds a new release notes page component and routing
- Implements a dismissible page banner with internationalized messaging
- Adds environment variable configuration for enabling/disabling the banner
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/release-notes/messages.js | Defines i18n messages for banner text and link text |
| src/release-notes/ReleaseNotes.scss | Styles for the page banner with custom colors and typography |
| src/release-notes/ReleaseNotes.jsx | Basic release notes page component |
| src/index.scss | Imports the new SCSS file |
| src/index.jsx | Adds environment variable configuration for banner control |
| src/course-outline/CourseOutline.tsx | Integrates the banner into the course outline page |
| src/CourseAuthoringRoutes.jsx | Adds routing for the release notes page |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
c8776fe to
eda0983
Compare
eda0983 to
5209143
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Ticket: TNL2-370
In this PR,
Screen.Recording.2025-10-06.at.6.31.23.PM.mov