Fix broken bulkEditTags
and refactor logic of the component
#1137
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.
Changes proposed ✍️
What
🤖 Generated by Copilot at ce43748
The pull request refactors the frontend code for bulk updating tags of members. It replaces the old
AppMemberListBulkUpdateTags
component with a newAppTagPopover
component that uses a form schema and a member store. It also moves thetag-popover
component to themember
module and updates its logic. It comments out a redundant popover component in themember-list-table
component and adds a TODO comment.
🤖 Generated by Copilot at ce43748
Why
How
🤖 Generated by Copilot at ce43748
tag-popover
component fromtag
module tomember
module (link)app-member-list-bulk-update-tags
component withapp-tag-popover
component inmember-list-toolbar
component (link, link)app-member-list-bulk-update-tags
component file (link)app-tag-popover
component inmember-list-table
component and add TODO comment (link)tag-popover
component to handle bulk update of tags for selected members (link, link, link, link, link, link)Checklist ✅
Feature
,Improvement
, orBug
.