Skip to content

Conversation

@djoseph-apphelix
Copy link
Member

@djoseph-apphelix djoseph-apphelix commented Oct 14, 2025

Description

  • Adds new functionality for managing release notes with a complete CRUD interface.

Key Changes

  • Integrated release notes into the Redux store with API integration
  • Created UI components for creating, editing, and viewing release notes
  • Added modal dialogs, sidebar navigation, and forms
  • Implemented internationalization with proper message definitions
  • Controlled feature visibility using the ENABLE_RELEASE_NOTES flag in environment files
  • Restricted create/edit/delete actions to administrator users using getAuthenticatedUser

Jira

Authors

Course.Authoring._.Your.Plaform.Name.Here.-.Google.Chrome.2025-10-21.15-38-50.mp4

@Copilot Copilot AI review requested due to automatic review settings October 14, 2025 10:36
Copy link

Copilot AI left a 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-commenter
Copy link

codecov-commenter commented Oct 14, 2025

Codecov Report

❌ Patch coverage is 67.30159% with 103 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.38%. Comparing base (5778876) to head (0a2a85e).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/release-notes/ReleaseNotes.jsx 0.00% 64 Missing ⚠️
src/release-notes/update-form/ReleaseNoteForm.jsx 65.00% 21 Missing ⚠️
src/release-notes/hooks.jsx 69.64% 17 Missing ⚠️
src/release-notes/delete-modal/DeleteModal.jsx 90.90% 1 Missing ⚠️
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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Copilot Copilot AI review requested due to automatic review settings October 16, 2025 05:48
Copy link

Copilot AI left a 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.

@Copilot Copilot AI review requested due to automatic review settings October 17, 2025 10:12
Copy link

Copilot AI left a 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.

@Copilot Copilot AI review requested due to automatic review settings October 21, 2025 05:49
Copy link

Copilot AI left a 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.

@Copilot Copilot AI review requested due to automatic review settings October 21, 2025 09:55
Copy link

Copilot AI left a 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.

Copy link

Copilot AI left a 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.

Comment on lines +117 to +128
.edit-notes-hover:hover{
background-color: #D3D3D3;
}

.delete-notes-hover:hover{
Copy link

Copilot AI Oct 21, 2025

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.

Suggested change
.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.

Copy link
Collaborator

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.

Comment on lines +117 to +128
.edit-notes-hover:hover{
background-color: #D3D3D3;
}

.delete-notes-hover:hover{
Copy link

Copilot AI Oct 21, 2025

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.

Suggested change
.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.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@djoseph-apphelix same here.

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.

4 participants