Closed (fixed)
Project:
Drupal core
Version:
11.x-dev
Component:
announcements_feed.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
26 Apr 2023 at 09:34 UTC
Updated:
30 Jun 2023 at 12:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
fjgarlin commentedComment #3
fjgarlin commentedComment #5
fjgarlin commentedReady to review.
Enable the module, then navigate to "/admin", and you should see something like this if you have toolbar enabled:

If the toolbar module is disabled, you can still see the link and access notifications.
If you navigate to the "/admin/help" page, and then click on "Announcements" (/admin/help/announcements_feed), you will also see a link to the announcements page.
Comment #6
smustgrave commentedWonder if the IS could be updated if a solution.
Tried testing the patch and if the solution to add a link to the Structure page I did not see the link. Cleared cache several times
And when toolbar module is enabled I'm not seeing Announcements twice in the toolbar. Think that could be confusing.
Comment #7
fjgarlin commentedComment #8
fjgarlin commentedI updated the IS with the solution that this MR provides.
It was never under the "Structure" page, it was, and it still is, under the "/admin" page. I tried to think about arguments to add it inside the "Content", "Structure" or "Configuration" menu, but this is none of those really, so a link under the "Administration" page seems appropriate.
That has been addressed now. It's shown only once.
--
With this MR, the help text provides a link to access the announcements page directly and it also shows a link under the "/admin" and "/admin/index" pages to access the announcements.
Comment #9
varun verma commentedI have review that, when toolbar module is enabled then Announcements showing once only. screenshot attached for reference.
Comment #10
smustgrave commentedThanks for the issue summary update.
Tested again when the toolbar module is disabled the link does appear under
/admin technically it appears with or without the toolbar module I should say.
Comment #14
longwaveDiscussed with @lauriii and @catch, given that this makes the new module more usable without toolbar being installed we agreed to backport to 10.1.x.
Committed and pushed 8b92bea8c7 to 11.x (10.2.x) and 611cda6107 to 10.1.x. Thanks!