Problem/Motivation

The new Announcements feed (created as part of the 'Project messaging in core initiative') is being added to Drupal core as an experimental module, using a toolbar based feed user interface.

However, in the time that this initiative has been underway, a new proposal for a modern dashboard has been developed: #3244581: Enhance user experience with customizable dashboards

Proposed resolution

As the dashboard concept is developed, move the announcements UI elements from the toolbar into a block on the dashboard.

Remaining tasks

  • Wait for initial development of dashboard concept
  • Migrate feed display into dashboard content (likely a block?)
  • Check caching and cron behaviors and make sure they are integrated with similar patterns for other dashboard items
  • Full usability review
  • Full accessibility review

User interface changes

  • Announcement content moves from a toolbar to a block on the dashboard
  • The 'notification' bell should probably remain, but perhaps might be superceded by a notification bell for any changes to the dashboard, not just to this feed.

API changes

n/a

Data model changes

n/a

Release notes snippet

The Drupal Announcements feed has been moved from a toolbar into the new Drupal dashboard interface. [TBA]

CommentFileSizeAuthor
#9 Screenshot 2023-10-23 at 10.23.15.png152.77 KBfjgarlin

Issue fork drupal-3349394

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

hestenet created an issue. See original summary.

hestenet’s picture

hestenet
He/Him
Location Portland, OR 🇺🇸
commented
Related issues: +#3244581: Enhance user experience with customizable dashboards
hestenet’s picture

hestenet
He/Him
Location Portland, OR 🇺🇸
commented
Status: Active » Postponed

Postponed on further progress on: #3244581: Enhance user experience with customizable dashboards

Version: 10.1.x-dev » 11.x-dev

Drupal core is moving towards using a “main” branch. As an interim step, a new 11.x branch has been opened, as Drupal.org infrastructure cannot currently fully support a branch named main. New developments and disruptive changes should now be targeted for the 11.x branch, which currently accepts only minor-version allowed changes. For more information, see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

plopesc’s picture

plopesc
he/him
Primary language Spanish
commented

In order to integrate announcements feed into the dashboard, #3375851: Provide a block with DA announcements was created.

It provides a block and some tests in the attached patch.

Would be great to include this new block as part of the announcements module.

That would be the 1st step to integrate it within dashboards as soon as they are available.

What do you think?

fjgarlin’s picture

fjgarlin commented

I'll comment on the issue. Thanks for linking it here.

plopesc opened merge request !5073

plopesc’s picture

plopesc
he/him
Primary language Spanish
commented
Status: Postponed » Needs review

Thank you for the quick response!

Created MR with 1st approach to include the Announcements block in the module.

Thanks!

fjgarlin’s picture

fjgarlin commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screenshot 2023-10-23 at 10.23.15.png152.77 KB

This is a great refactoring into a reusable service, leaving both the controller and the block definition in a very clean way. Thanks for adding additional tests as well for the new service and the block. The new changes make the announcements be available as a block as well.

I reviewed the code and couldn't find anything to report, so it looks good from my end. I also tested the new block via the UI using block layout and it looks good. I tested logged in and logged out too.
block

Tests are passing for both DrupalCI and GitLabCI.

Marking this RTBC.

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
commented
Title: Convert announcements feed from toolbar to dashboard » Provide announcement feed as a block
Status: Reviewed & tested by the community » Needs work

Posted a minor nitpick on the MR. Feel free to move back to RTBC after addressing the feedback.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented

We should provide credit also to friera, who worked initially on this as part of the dashboard initiative: https://www.drupal.org/project/3327580/issues/3375851

fjgarlin’s picture

fjgarlin commented
Status: Needs work » Reviewed & tested by the community

RTBC based on #10 as the changes were addressed.

  • lauriii committed 529c7932 on 11.x 
    Issue #3349394 by plopesc, fjgarlin, hestenet, penyaskito, friera:...

  • lauriii committed 0229be4a on 10.2.x 
    Issue #3349394 by plopesc, fjgarlin, hestenet, penyaskito, friera:...

lauriii credited friera.

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
commented
Version: 11.x-dev » 10.2.x-dev
Status: Reviewed & tested by the community » Fixed

Committed 529c793 and pushed to 11.x. Also cherry-picked to 10.2.x. Thanks!

lauriii closed merge request !5073

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.