-
Notifications
You must be signed in to change notification settings - Fork 0
feat: new pages for release notes #6
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
base: master
Are you sure you want to change the base?
Conversation
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
Adds new functionality for managing release notes with a complete CRUD interface. This includes Redux state management for release notes data, UI components for creating/editing/viewing release notes, and comprehensive test coverage.
- Integrates release notes into the Redux store and implements data layer with API integration
- Creates comprehensive UI components including forms, sidebar navigation, and modal dialogs
- Implements internationalization support with proper message definitions
Reviewed Changes
Copilot reviewed 22 out of 22 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/store.ts | Adds releaseNotesReducer to Redux store configuration |
| src/release-notes/update-form/unsaved-modal-messages.js | Defines i18n messages for unsaved changes modal |
| src/release-notes/update-form/ReleaseNoteForm.test.jsx | Test coverage for release note form component |
| src/release-notes/update-form/ReleaseNoteForm.jsx | Main form component for creating/editing release notes |
| src/release-notes/sidebar/ReleaseNotesSidebar.test.jsx | Test coverage for sidebar navigation component |
| src/release-notes/sidebar/ReleaseNotesSidebar.jsx | Sidebar component for release note navigation |
| src/release-notes/messages.js | Adds comprehensive i18n message definitions |
| src/release-notes/hooks.test.jsx | Test coverage for release notes custom hook |
| src/release-notes/hooks.jsx | Custom hook managing release notes state and actions |
| src/release-notes/delete-modal/messages.js | i18n messages for delete confirmation modal |
| src/release-notes/delete-modal/DeleteModal.test.jsx | Test coverage for delete modal component |
| src/release-notes/delete-modal/DeleteModal.jsx | Delete confirmation modal component |
| src/release-notes/data/thunk.test.js | Test coverage for Redux thunk actions |
| src/release-notes/data/thunk.js | Redux thunk implementations for API operations |
| src/release-notes/data/slice.test.js | Test coverage for Redux slice |
| src/release-notes/data/slice.js | Redux slice managing release notes state |
| src/release-notes/data/selectors.test.js | Test coverage for Redux selectors |
| src/release-notes/data/selectors.js | Redux state selectors |
| src/release-notes/data/api.test.js | Test coverage for API layer |
| src/release-notes/data/api.js | API functions for release notes CRUD operations |
| src/release-notes/ReleaseNotes.scss | Styling for release notes components |
| src/release-notes/ReleaseNotes.jsx | Main release notes page component |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #6 +/- ##
==========================================
- Coverage 94.69% 94.38% -0.32%
==========================================
Files 1204 1214 +10
Lines 26854 27167 +313
Branches 6025 6134 +109
==========================================
+ Hits 25430 25642 +212
- Misses 1354 1455 +101
Partials 70 70 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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 25 out of 25 changed files in this pull request and generated 10 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
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 30 out of 30 changed files in this pull request and generated 6 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
5511c1f to
64a83fe
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
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 30 out of 30 changed files in this pull request and generated 7 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| .edit-notes-hover:hover{ | ||
| background-color: #D3D3D3; | ||
| } | ||
|
|
||
| .delete-notes-hover:hover{ |
Copilot
AI
Oct 21, 2025
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.
[nitpick] Missing space before the opening brace. Should be .edit-notes-hover:hover { to follow standard CSS formatting conventions.
| .edit-notes-hover:hover{ | |
| background-color: #D3D3D3; | |
| } | |
| .delete-notes-hover:hover{ | |
| .edit-notes-hover:hover { | |
| background-color: #D3D3D3; | |
| } | |
| .delete-notes-hover:hover { |
Copilot uses AI. Check for mistakes.
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.
@djoseph-apphelix can we add spaces here.
| .edit-notes-hover:hover{ | ||
| background-color: #D3D3D3; | ||
| } | ||
|
|
||
| .delete-notes-hover:hover{ |
Copilot
AI
Oct 21, 2025
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.
[nitpick] Missing space before the opening brace. Should be .delete-notes-hover:hover { to follow standard CSS formatting conventions.
| .edit-notes-hover:hover{ | |
| background-color: #D3D3D3; | |
| } | |
| .delete-notes-hover:hover{ | |
| .edit-notes-hover:hover { | |
| background-color: #D3D3D3; | |
| } | |
| .delete-notes-hover:hover { |
Copilot uses AI. Check for mistakes.
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.
@djoseph-apphelix same here.
…in access control
dbccbb0 to
a60ce10
Compare
Description
Key Changes
ENABLE_RELEASE_NOTESflag in environment filesJira
Authors
Course.Authoring._.Your.Plaform.Name.Here.-.Google.Chrome.2025-10-21.15-38-50.mp4