Skip to content

feat: Inline editing - usergroup - #1978

Merged
enjeck merged 2 commits into
mainfrom
feat/364-usergroup
Aug 18, 2025
Merged

feat: Inline editing - usergroup#1978
enjeck merged 2 commits into
mainfrom
feat/364-usergroup

Conversation

@enjeck

@enjeck enjeck commented Aug 5, 2025

Copy link
Copy Markdown
Contributor

Part of #364

Supports inline editing for the usergroup column

TODO

  • find alternative to setTimeout

@enjeck enjeck self-assigned this Aug 5, 2025
@enjeck
enjeck requested a review from blizzz as a code owner August 5, 2025 06:47
@enjeck enjeck added the 3. to review Waiting for reviews label Aug 5, 2025
@github-project-automation github-project-automation Bot moved this to 馃Л Planning evaluation (don't pick) in 馃摑 Productivity team Aug 5, 2025
@enjeck

enjeck commented Aug 5, 2025

Copy link
Copy Markdown
Contributor Author

Here's how it works:
Screencast from 2025-08-05 07-46-31.webm

@enjeck
enjeck requested a review from marcoambrosini August 5, 2025 06:48
@enjeck enjeck mentioned this pull request Aug 5, 2025
17 tasks
Base automatically changed from feat/364 to main August 5, 2025 07:29
marcoambrosini

This comment was marked as resolved.

@enjeck
enjeck force-pushed the feat/364-usergroup branch from 0a23bca to 994fbfa Compare August 14, 2025 01:05
@enjeck
enjeck requested a review from silverkszlo August 14, 2025 08:19
Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>
@enjeck
enjeck force-pushed the feat/364-usergroup branch from 994fbfa to 09078ce Compare August 17, 2025 18:17
Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>
Comment thread src/shared/components/ncTable/partials/TableCellUsergroup.vue
Comment thread src/shared/components/ncTable/partials/TableCellUsergroup.vue

@silverkszlo silverkszlo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's nice! Would also be great to make this one more accessible for keyboard usage.

@enjeck
enjeck merged commit 0e205e9 into main Aug 18, 2025
50 of 52 checks passed
@enjeck
enjeck deleted the feat/364-usergroup branch August 18, 2025 14:55
@github-project-automation github-project-automation Bot moved this from 馃Л Planning evaluation (don't pick) to 鈽戯笍 Done in 馃摑 Productivity team Aug 18, 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