SHS-5981: Site Dashboard: Active Site Editors

[mariannuar]

Summary

Add an Active Site Editors block for the Dashboard. This ticket depends on #1719

Need Review By (Date)

01/24

Urgency

high

Steps to Test

1. Go to /admin/structure/dashboard or https://pr1721-owc6uhbjbocdblzr55v0je8nvbwiunyi.tugboatqa.com/admin/dashboard
2. Create a dashboard if there isn't an existing one
3. Enable it
4. Edit the layout
5. Create a section
6. Add a block, confirm that now you can add one that is called Active Site Editors and add it
7. Now go to /admin/dashboard or the preview of the dashboard
8. Confirm the Active Site Editors block looks like this

9. The block should show the username, roles and the last access

Review Tasks

Backend / Functional Validation

Code

• Are the naming conventions following our standards?
• Are PHP functions and variables in snake_case and not camelCase?
• Does Drupal code follow Drupal Coding Standards?
• Does the code have sufficient inline comments?
• Is there anything in this code that would be hidden or hard to discover through the UI?
• Are there any code smells?
• Are tests provided?

Code security

• Is all user input properly sanitized when rendered?
• Any obvious security flaws or new areas for attack?

General

• Is there anything included in this PR that is not related to the problem it is trying to solve?
• Is the approach to the problem appropriate?

[cienvaras]

@mariannuar Looks good, however the "View all active site editors" don't match the AC, please check the ticket and adjust accordingly.

Also, the previous version of this PR had some CSS. Things look good on the dashboard, so I just wanted to confirm if it's not needed anymore or if you removed it by accident.

[mariannuar]

@cienvaras This is ready for review again! https://pr1721-owc6uhbjbocdblzr55v0je8nvbwiunyi.tugboatqa.com/admin/dashboard

And yeah, there were some CSS styles, but I figured it out that since those styles were going to be the same for this kind of blocks (with tables), I moved those styles to the other PR and make it more general. I don't think the styles will change a lot for each block, or that's not what I understood in the design Tori provided in the other ticket of design implementation.

[cienvaras]

@mariannuar LGTM!

@ahughes3 Ready for you.

[cienvaras]

Merging into #1719 to make it easier to review the dashboard base design.