Skip to content

Conversation

@ekes
Copy link
Member

@ekes ekes commented Aug 18, 2025

Permissions applied the same as the would be to existing sites by the module upgrade.

What does this change?

Adds new permissions introduced in Group Invite 4.0.0.

How to test

There's a failing test to visit the invites page. This should now pass.

Permissions applied the same as the would be to existing sites by the
module upgrade.
@ekes
Copy link
Member Author

ekes commented Aug 18, 2025

The PHP8.1 test installation failure is a new one on me. Someone want to check this?

We're kinda roadblocked with this needed for: #542 and that needed for localgovdrupal/localgov_microsites_project#79

@stephen-cox @millnut maybe?

@ekes ekes requested review from millnut and stephen-cox August 18, 2025 11:10
@tonypaulbarker
Copy link

tonypaulbarker commented Aug 19, 2025

@ekes I gave this a look but I couldn't see how to check permissions are applied correctly. Filtering permissions for 'invite' doesn't return any rows.

Is there some place we can check group permissions and roles?

Screenshot 2025-08-19 at 11 05 50

@millnut
Copy link
Member

millnut commented Aug 19, 2025

@ekes could you try clearing the cache, 8.1 sees the env var for LATEST_RELEASE as 1 which indicates something is not working correctly when getting/setting env vars, whereas the others have the correct 4.0.5 version

@ekes
Copy link
Member Author

ekes commented Aug 20, 2025

@tonypaulbarker

Is there some place we can check group permissions and roles?

Group permissions are defined on the group bundle: admin/group/types/manage/microsite/permissions

image --- image

@ekes
Copy link
Member Author

ekes commented Aug 20, 2025

@millnut

could you try clearing the cache

Looks like that did it. Thanks.

@ekes ekes requested review from tonypaulbarker and removed request for millnut August 20, 2025 13:30
@tonypaulbarker tonypaulbarker merged commit 92e19ce into 4.x Aug 26, 2025
13 of 14 checks passed
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.

4 participants