Dev #1257
Closed
Dev #1257
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Implement edit custom section functionality: - Add EditCustomSectionModal with validation for required metadata - Call updateCustomSection mutation to persist changes - Navigate back to custom section details page after editing - Prevent selecting past dates for start and end dates - Allow start and end dates to be the same day - Ensure end date is not before start date Implement section item management: - Add ability to manually add items to sections from custom section details page - Add ability to add items to sections from corpus page via schedule form - Call createSectionItem mutation when adding items - Support adding both existing and new corpus items - Filter out expired sections from section dropdown - Add status labels (Scheduled/Live) to sections in dropdown Implement section deletion: - Add delete section functionality with confirmation modal - Call deleteCustomSection mutation for soft delete - Replace window.confirm with DeleteConfirmationModal component - Update tests to work with modal-based confirmation Additional improvements: - Replace hardcoded hex colors with curationPalette references - Add border, lightRed, and veryLightGrey to theme palette - Extract onSave handler into separate method for readability - Remove incorrect default values (topic: 'TECHNOLOGY') from form transformations - Use empty string defaults for controlled inputs to prevent React warnings - Fix prettier formatting throughout
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Goal
Insert purpose of pull request
Todos
Modifying Environment Variables
Omit this section if not applicable.
Have you followed the guidelines in our Contributing document?
Deployment
Omit this section if not applicable.
Anything we should know about deploying this PR?
Reference
Documentation here:
Tickets:
Implementation Decisions