Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #3470128: Make quick join configurable for custom group types #4046

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ronaldtebrake
Copy link
Contributor

Problem && Solution

We used to load all roles / group types based on if -outsider is part of the name. This worked fine for a lot of the circumstances of the old group type where this was the case. Now, with the move to Group v2, oustider became a scope of the actual group role. So to make this compatible, we load all group roles, and figure out if we have a role with outsider scope that has the permission to join groups. Any group types without an outsider scope and or the join group permission we can't configure. As that means there is no group role specified that is able to quickjoin.

Issue tracker

https://www.drupal.org/project/social/issues/3470128

How to test

  • Create custom group type, with outsider scoped roles, that don't have -outsider as part of the group name.
  • See that the quick join page doesn't give an error
  • See that all the group types are showing with an outsider role that has join group

Release notes

Make sure quick join group has all group types available, including custom ones.

@ronaldtebrake ronaldtebrake added type: bug Fixes a bug in Open Social status: needs review This pull request is waiting for a requested review prio: high team: guardians team: iata labels Aug 25, 2024
@ronaldtebrake ronaldtebrake added this to the 13.0.0-alpha10 milestone Aug 25, 2024
@ronaldtebrake ronaldtebrake changed the title Allow custom group types to be configured. Issue #3470128: Allow custom group types to be configured for quick join settings. Aug 25, 2024
@ronaldtebrake ronaldtebrake changed the title Issue #3470128: Allow custom group types to be configured for quick join settings. Issue #3470128: Make quick join configurable for custom group types Aug 25, 2024
We used to load all roles / group types based on if -outsider is part of the name.
This worked fine for a lot of the circumstances of the old group type where this was the case.
Now, with the move to Group v2, oustider became a scope of the actual group role.
So to make this compatible, we load all group roles, and figure out if we have a role with outsider scope that has
the permission to join groups. Any group types without an outsider scope and or the join group permission we can't configure.
As that means there is no group role specified that is able to quickjoin.
@ronaldtebrake ronaldtebrake force-pushed the issue/PROD-30438-quick-join-custom-group-types branch from 5118c1b to 4307d7e Compare August 25, 2024 08:42
@open-social-tugboat
Copy link

Tugboat has finished building the preview for this pull request!

Link:

Dashboard:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
prio: high status: needs review This pull request is waiting for a requested review team: guardians type: bug Fixes a bug in Open Social
Development

Successfully merging this pull request may close these issues.

5 participants