[WEB-2237] fix: intake issue create/update modal#5434
Conversation
WalkthroughThe changes to the Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant CustomMenu
participant ParentIssueHandler
User->>CustomMenu: Open menu
CustomMenu->>User: Display options (Change, Remove)
User->>CustomMenu: Select Change
CustomMenu->>ParentIssueHandler: Update parent issue
ParentIssueHandler->>User: Confirm update
User->>CustomMenu: Select Remove
CustomMenu->>ParentIssueHandler: Remove parent issue
ParentIssueHandler->>User: Confirm removal
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- web/core/components/inbox/modals/create-edit-modal/issue-properties.tsx (4 hunks)
Additional comments not posted (4)
web/core/components/inbox/modals/create-edit-modal/issue-properties.tsx (4)
5-5: LGTM!The import statement for
CustomMenuis correct and necessary.
98-98: LGTM!The updated
onChangehandler for the start date improves code clarity and ensures consistent data handling.
110-110: LGTM!The updated
onChangehandler for the due date improves code clarity and ensures consistent data handling.
161-197: LGTM! But verifyhandleDatafunction.The
CustomMenucomponent enhances the user interface by providing interactive options for managing parent issues. Ensure that thehandleDatafunction properly handles empty strings when removing a parent issue.Run the following script to verify the
handleDatafunction:
Changes:
This PR fixes a bug in the intake create/update modal where users couldn't unselect the start date, due date, or parent once they were selected. I've made the necessary changes to resolve this issue.
Reference:
[WEB-2237]
Summary by CodeRabbit
New Features
CustomMenufor selecting and managing parent issues, improving user experience.Improvements
DateDropdowncomponents for better clarity and consistency.