Skip to content

feat: Add roles for users within backend - #2331

Draft
sh-ran wants to merge 2 commits into
activist-org:mainfrom
sh-ran:roles
Draft

sh-ran wants to merge 2 commits into
activist-org:mainfrom
sh-ran:roles

Conversation

@sh-ran

@sh-ran sh-ran commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

Contributor checklist


Description

This PR adds roles for users within the backend. These roles will be used to check/obtain permissions within the backend for GET, POST, PUT and DELETE api calls done by users.

Related issue

@netlify

netlify Bot commented Aug 8, 2026

Copy link
Copy Markdown

Deploy Preview for activist-org ready!

Name Link
🔨 Latest commit cf04f61
🔍 Latest deploy log https://app.netlify.com/projects/activist-org/deploys/6a9c3ba7c357100008b28c59
😎 Deploy Preview https://deploy-preview-2331--activist-org.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@andrewtavis andrewtavis added the -priority- High priority label Aug 8, 2026
@andrewtavis
andrewtavis marked this pull request as draft August 8, 2026 17:26
@andrewtavis

Copy link
Copy Markdown
Member

note: We're switching this to a draft as @sh-ran still has some work to do to clean the implementation up :)

@andrewtavis andrewtavis added the backend Relates to the project backend label Aug 12, 2026
@aasimsyed

Copy link
Copy Markdown
Member

Just flagging one thing early since it affects everything downstream: role is added alongside the existing is_owner/is_admin/is_comms booleans, and CI is currently failing because the new unique_org_membership/unique_group_membership constraints conflict with existing tests that create multiple membership rows per (org/group, user).

Does role fully replace the booleans (needs a migration/backfill), or coexist for a while?

@andrewtavis

Copy link
Copy Markdown
Member

Thanks for noting this, @aasimsyed! Depends on the speed here. Might replace if we can get this done in say the next two weeks to a month.

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

Labels

-priority- High priority backend Relates to the project backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add user roles for organizations in backend and frontend

3 participants