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

Set if particular content types can be created in different Microsites #428

Open
ekes opened this issue Oct 20, 2023 · 4 comments
Open

Set if particular content types can be created in different Microsites #428

ekes opened this issue Oct 20, 2023 · 4 comments

Comments

@ekes
Copy link
Member

ekes commented Oct 20, 2023

I'll be honest here. My temptation is now to remove group_permissions and use custom entity access to set if content can be created - it will then always be possible to be viewed if it has been created.

Originally posted by @ekes in #427 (comment)

@finnlewis
Copy link
Member

Good idea. I wonder how much time this would take?

If we can estimate the size of the job and upgrade paths, we can assess.

@ekes
Copy link
Member Author

ekes commented Oct 29, 2023

Well the migration to a field, supplied with modules, and a new hook... straightforward. I've done a fair bit when prodding.

However I am reminded why going for group_permissions in the first place was more 'straightforward'. As the group routes to create content, and probably other custom /group/{group}/... routes, use the plugin access which doesn't look at the entity_access hooks at all.

So we'd do entity access hooks and add additional access checks on specific routes or decorate the access service plugin further -- thinking about it the later might be best. But, it'd probably still be better for performance rather than entering the calculator on a per-group basis as group_permissions does, and it'd be 'simpler'. So probably still worth it for what we need which is the impression of particular content types per site and simple ui without the additional types.

@ekes
Copy link
Member Author

ekes commented May 7, 2024

  • Switch to use entity access checks per group.
  • Optional: see if this can be better done with a permissions service per group domain context
  • Add more tests to ensure this is working correctly.

@ekes ekes changed the title Proposal: Remove group_permissions and use custom entity access to set if content can be created in a Microsite Remove group_permissions and use custom entity access to set if content can be created in a Microsite May 7, 2024
@ekes ekes changed the title Remove group_permissions and use custom entity access to set if content can be created in a Microsite Set if content can be created in a Microsite May 7, 2024
@ekes ekes changed the title Set if content can be created in a Microsite Set if particular content types can be created in different Microsites May 7, 2024
@finnlewis
Copy link
Member

Added this issue to write some tests: #464

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

No branches or pull requests

2 participants