feat(bulk-edit): sectioned field list with A–Z sorting in flyout - #46
Merged
Merged
Conversation
Organize the Edit fields flyout into Recent, Issue properties, Project fields, and Relationships sections with alphabetical ordering in browse mode. Flatten to a single match list when searching, wire relationship editing through a dedicated pane, and extend tests for partitioning and relationship payloads.
There was a problem hiding this comment.
Cursor auto review
Found 2 actionable issue(s) on changed lines.
Solid sectioned field list and relationship pane wiring, but two correctness gaps: flyout state is not cleared when the parent closes via Escape/selection clear, and relationship editor state bleeds across field switches.
Generated automatically when this PR was submitted using Cursor CLI with --model auto.
Contributor
There was a problem hiding this comment.
4 issues found across 7 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
Reset flyout state on prop-driven close, guard preflight races, and return actionable errors when relationship validation or apply throws.
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
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.
Summary
Changes
Flyout UI (
src/features/bulk-edit-flyout.tsx)Field catalog (
src/features/bulk-edit-utils.tsx,src/features/bulk-edit-relationship-pane.tsx)partitionFieldList, relationship field IDs, and catalog builders for browse vs search modesBar integration (
src/features/bulk-actions-bar.tsx,src/features/bulk-actions-utils.tsx)repoNameinto the flyout for metadata and relationship searchbuildRelationshipsPayloadfor background message dispatchTests (
src/features/__tests__/bulk-edit-flyout.test.tsx)Test Plan
pnpm vitest run src/features/__tests__/bulk-edit-flyout.test.tsxSummary by cubic
Reorganized the Edit fields flyout into sectioned lists with A–Z sorting and added a dedicated relationships editor (parent/blocked-by/blocking). Search now shows a single, deduped list, and the apply flow is more robust with state resets, debounced validation, and clearer errors.
New Features
Commentis supported.repoNameinto the flyout; new utils for field catalog/partitioning andbuildRelationshipsPayload; tests cover sorting, search dedupe,Commentapply gating, and relationship payloads.Bug Fixes
Written for commit 55228a4. Summary will update on new commits.