Problem/Motivation

Currently there's the option to exclude logged in users. The more flexible Drupal-way would be to include / exclude roles or even better use Conditions API like we for example did in the Posthog Module:
https://www.drupal.org/project/posthog

Also see https://www.drupal.org/project/drupal/issues/3506721

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments

anybody created an issue. See original summary.

firflant’s picture

firflant
Location Poland
commented

I wonder how big impact it will add to the site load in comparison to authenticated role checker. I would like to keep this module lightweight

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Related issues: +#3571787: Add a textarea to exclude certain paths (with useful defaults)

Yeah absolutely your choice as maintainer :)

Still a role selection would be more typical than just authenticated.
The "exclude on admin paths" option should be separate from that, I think and should use https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Routing%2... technically IMHO.

firflant’s picture

firflant
Location Poland
commented
Status: Active » Closed (won't fix)

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.