Skip to content

fix: Add key to pre-existing usergroup items - #2117

Merged
enjeck merged 3 commits into
mainfrom
add-key-to-usergroup
Nov 11, 2025
Merged

fix: Add key to pre-existing usergroup items#2117
enjeck merged 3 commits into
mainfrom
add-key-to-usergroup

Conversation

@enjeck

@enjeck enjeck commented Oct 23, 2025

Copy link
Copy Markdown
Contributor

To reproduce the problem:

  • Create a table and add a usergroup column with "multiple" turned on
  • Add a new row with multiple users and save
  • Try to remove one of the users/groups. All of the users get cleared

This happens because there's no key associated with the item. So, we therefore add keys to the pre-existing usergroup items. With keys, only the clicked item is cleared.

@enjeck enjeck self-assigned this Oct 23, 2025
@enjeck
enjeck requested a review from blizzz as a code owner October 23, 2025 04:51
@enjeck enjeck added the 3. to review Waiting for reviews label Oct 23, 2025
@github-project-automation github-project-automation Bot moved this to 馃Л Planning evaluation (don't pick) in 馃摑 Productivity team Oct 23, 2025
@enjeck
enjeck force-pushed the add-key-to-usergroup branch from 0a631ca to 78023c9 Compare November 7, 2025 10:06
@enjeck
enjeck force-pushed the add-key-to-usergroup branch 2 times, most recently from 76b5338 to eaf08e2 Compare November 11, 2025 19:26
Signed-off-by: Enjeck C <patrathewhiz@gmail.com>
Signed-off-by: Enjeck C <patrathewhiz@gmail.com>
Signed-off-by: Enjeck C <patrathewhiz@gmail.com>
@enjeck
enjeck force-pushed the add-key-to-usergroup branch from eaf08e2 to 05b087f Compare November 11, 2025 19:26
@enjeck
enjeck merged commit 0ea7f55 into main Nov 11, 2025
53 checks passed
@enjeck
enjeck deleted the add-key-to-usergroup branch November 11, 2025 19:44
@github-project-automation github-project-automation Bot moved this from 馃Л Planning evaluation (don't pick) to 鈽戯笍 Done in 馃摑 Productivity team Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants