Skip to content

Restrict groups - #539

Open
bnmajor wants to merge 9 commits into
masterfrom
restrict-groups
Open

Restrict groups#539
bnmajor wants to merge 9 commits into
masterfrom
restrict-groups

Conversation

@bnmajor

@bnmajor bnmajor commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

No description provided.

bnmajor added 7 commits August 5, 2026 10:44
Assign `originalAnnotation` on the model before it enters the collection so the menu reflects the
clicked annotation rather than the panel selection.
…tion

- Context menu auto-creates a restricted annotation's allowed groups the first time it
  encounters them
- Regression test asserting context menu renders clicked annotation's restriction on selection
  rather than the active annotation in the panel
@bnmajor
bnmajor marked this pull request as ready for review August 5, 2026 15:25
@bnmajor
bnmajor requested a review from manthey August 5, 2026 15:25
@manthey

manthey commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

We'll need to add to the documentation somewhere for this, too.

@manthey

manthey commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

I guess we should add a new file to the docs directory (and, as a separate PR, we really should have links to the docs in the docs directory from somewhere).

This works well in my testing.

I wonder if we should accept both a proper JSON array AND a flat string (e.g., "allowed_groups": ["Blue","Yellow"] or "allowed_groups": '["Blue","Yellow"]') so that users don't have to "json-ize" strings in the UI metadata editor .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants