Skip to content

YDA-6550: disable demoting yourself as group manager to viewer#433

Merged
claravox merged 2 commits intodevelopmentfrom
YDA-6550-demote-to-viewer
Sep 22, 2025
Merged

YDA-6550: disable demoting yourself as group manager to viewer#433
claravox merged 2 commits intodevelopmentfrom
YDA-6550-demote-to-viewer

Conversation

@claravox
Copy link
Contributor

@claravox claravox commented Sep 19, 2025

Current policies demote user in two steps. This causes internal error for demoting yourself as a group manager. This is a workaround to prevent the user from doing this.

I tried to use less jquery-dependent way.

Relates to UtrechtUniversity/yoda#612

Current policies demote user in two steps. This causes internal error for demoting yourself as a group manager. This is a workaround to prevent the user from doing this.
@claravox claravox force-pushed the YDA-6550-demote-to-viewer branch from f3cae4d to 71a09f7 Compare September 19, 2025 13:56
@claravox claravox marked this pull request as ready for review September 19, 2025 14:20
Copy link
Contributor

@leonidastri leonidastri left a comment

Choose a reason for hiding this comment

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

The logic looks good to me.

Copy link
Contributor

@claudisa-uu claudisa-uu left a comment

Choose a reason for hiding this comment

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

LGTM

@claravox claravox merged commit 56fafc9 into development Sep 22, 2025
10 checks passed
@claravox claravox deleted the YDA-6550-demote-to-viewer branch September 22, 2025 09:40
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.

3 participants