- Merge request !3565#3063877 - Add new permission for viewing the filter/tips page → (Open) created by danielveza
- Status changed to Needs review
about 2 years ago 4:42am 1 March 2023 - 🇳🇿New Zealand danielveza Brisbane, AU
Added a new permission "view filter tips page" for handling access to the generic /filter/tips page.
Will need a change record and IS update if this approach is signed off
- Status changed to Needs work
about 2 years ago 10:02am 1 March 2023 The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- 🇳🇿New Zealand quietone
quietone → changed the visibility of the branch 3063877-add-access-control to hidden.
- 🇮🇳India KumudB Ahmedabad
I have raised the MR for filter/tips page access permission.
MR : https://git.drupalcode.org/project/drupal/-/merge_requests/9617/diffs - 🇺🇸United States smustgrave
Proposed solution doesn't match what appears in the MR.
Also will need some kind of update hook to assign the new permission, without it people who correctly should have access all of a sudden won't.
New permissions will need a CR.
- 🇺🇸United States smustgrave
Also why start a new MR, the previous one 3565 is pointing to 11.x correctly
- Status changed to Needs review
about 2 months ago 11:44am 8 February 2025 - 🇳🇿New Zealand quietone
Unfortunately the MR added in #23 added an unnecessary change and caused a new failure. I've hidden that one and continued with the original 3565.
This adds fixes for some of the tests.
The new permission has been added to the authenticated role in Umami and Standard which needs to be discussed. The Filter module is not enabled in either of the profiles and that is causing test failures. It seems that Filter now needs to be installed in both profiles. And I would then only add the new permission, to the author, editor and content_editor roles.
- 🇬🇧United Kingdom catch
I'm wondering if we should remove filter tips altogether, it is not really fit for purpose. Sites can add description text to text fields with formatting hints when not using a WYSIWYG editor. Would need product manager sign off and probably in a new issue, with this one postponed in case it doesn't happen?
- 🇳🇿New Zealand quietone
OK. Let's no make a change here until that is decided.