Skip to content

feat: Allow assigning roles to creator#1216

Merged
RangerMauve merged 8 commits intomainfrom
feat/assign-role-to-creator
Feb 10, 2026
Merged

feat: Allow assigning roles to creator#1216
RangerMauve merged 8 commits intomainfrom
feat/assign-role-to-creator

Conversation

@RangerMauve
Copy link
Contributor

Closes #1206

  • Remove check for creator in assign role
  • Update test to check that assigning role to creator is possible
  • Add new test to check that creator can be re-added

Review wanted: Basic sanity check or suggestions for more tests

Copy link
Member

@achou11 achou11 left a comment

Choose a reason for hiding this comment

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

Changes overall make sense to me. Noticed a minor blocker that should be addressed before merging.

Is it expected that the new test takes a while? On my machine it takes around 45 seconds to finish...CI is stalled due to issues with GitHub right now, but would be interested to know how it performs there.

Co-authored-by: Andrew Chou <andrewchou@fastmail.com>
@RangerMauve
Copy link
Contributor Author

@achou11 The tests taking a while is due to #1217 Not sure if this should block the feature. 😅

@achou11
Copy link
Member

achou11 commented Feb 2, 2026

@achou11 The tests taking a while is due to #1217 Not sure if this should block the feature. 😅

Ah got it. To clarify, the blocker I was referring to was the leftover test.only that you've since addressed 😄 Agree that the issue probably shouldn't block this PR

@RangerMauve RangerMauve requested a review from achou11 February 2, 2026 21:38
achou11
achou11 previously approved these changes Feb 2, 2026
@RangerMauve RangerMauve merged commit 61f4077 into main Feb 10, 2026
11 checks passed
@RangerMauve RangerMauve deleted the feat/assign-role-to-creator branch February 10, 2026 19:43
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.

Creator should be able to be removed from project

3 participants