Skip to content

Global Groups: Database and group policy updates - #1800

Merged
ericenns merged 6 commits into
mainfrom
global-groups
May 11, 2026
Merged

Global Groups: Database and group policy updates#1800
ericenns merged 6 commits into
mainfrom
global-groups

Conversation

@deepsidhu85

@deepsidhu85 deepsidhu85 commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

What does this PR do and why?

Describe in detail what your merge request does and why.

This PR adds in the public column to the namespaces table, updates the group policy scope to return public groups, and updates the effective_access_level method to return a role of GUEST if the namespace is public and the user is not a member of the group or any groups the group was shared with

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other pull requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Login as any user and create a group
  2. In rails console, update the group to (public: true)
  3. Logout from IRIDA Next and login as a different user
  4. Verify they can see the group (only top level currently) with GUEST access

Note: Will be updated in the next PR to set group projects and subgroups as public where the parent group in public

PR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

@deepsidhu85 deepsidhu85 self-assigned this Apr 22, 2026
@deepsidhu85
deepsidhu85 force-pushed the global-groups branch 2 times, most recently from 7ba24f0 to afa8403 Compare April 23, 2026 15:44
@deepsidhu85
deepsidhu85 force-pushed the global-groups branch 2 times, most recently from 0465b0c to c1c8549 Compare May 5, 2026 15:23
Sukhdeep Sidhu added 5 commits May 8, 2026 14:03
…o include public groups, updated effective_access_level method to return GUEST role if they are not a member of the group or any group that the namespace has been shared with
@deepsidhu85
deepsidhu85 requested a review from ericenns May 11, 2026 13:11
@deepsidhu85
deepsidhu85 marked this pull request as ready for review May 11, 2026 13:11

@ericenns ericenns left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me!

@ericenns
ericenns merged commit 3683334 into main May 11, 2026
14 checks passed
@ericenns
ericenns deleted the global-groups branch May 11, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants