Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve display of centralized content on local content where it is embedded #4055

Open
kevwalsh opened this issue Jan 21, 2021 · 2 comments
Labels
Centralized Content A content type used across Facilties content Facilities Facilities products (VAMC, Vet Center, etc) Icebox Deprioritized for now Needs refining Issue status sitewide VAMC CMS managed product owned by Facilities team VBA for VBA facilities that are not Regional Offices Vet Center CMS managed product owned by Facilities team

Comments

@kevwalsh
Copy link
Contributor

kevwalsh commented Jan 21, 2021

Original ticket

As an editor of local content i can see the content moderation state of centralized content.

Notes by @davidmpickett

I'm not sure exactly what the original intent was here, but I do think there is a pretty good example of the pattern we should generally follow for Centralized Content embedded in local content that gets to the spirit of it.

On Vet Center nodes, there are several gray boxes where Centralized Content is displayed but can't be edited. When you go to edit this node, these gray boxes will get a clickable timestamp added to the lower right hand corner.

screencapture-prod-cms-va-gov-node-3783-edit-2023-09-18-17_54_54

Local editors can click through and see the current revision and a history of changes (including content moderation states).

screencapture-staging-cms-va-gov-centralized-content-national-vet-center-content-2023-09-18-18_07_08

Steps to reproduce

However, these clickable timestamps don't appear

  • In the View Node interface
  • In similar boxes on VAMCs top task pages (at least Medical Records doesn't - haven't checked them all)

screencapture-staging-cms-va-gov-node-45682-edit-2023-09-18-18_22_42

Possible path forward

  • Audit all Content Types that have an associated Centralized Content node
  • UX - propose a standardized way of linking from these gray boxes back to the relevant Centralized content
  • Dev - implement it retroactively on previous Content Types from audit
@kevwalsh kevwalsh added the Needs refining Issue status label Jan 21, 2021
@kevwalsh kevwalsh added the Icebox Deprioritized for now label Jan 27, 2022
@jilladams jilladams added VAMC CMS managed product owned by Facilities team Facilities Facilities products (VAMC, Vet Center, etc) Vet Center CMS managed product owned by Facilities team VBA for VBA facilities that are not Regional Offices Centralized Content A content type used across Facilties content labels Sep 1, 2023
@swirtSJW
Copy link
Contributor

I think the intent was to show pink if the centralized content node has never been published.

@davidmpickett davidmpickett changed the title Reveal content moderation status of centralized content on local content where it is embedded Improve display of centralized content on local content where it is embedded Sep 19, 2023
@swirtSJW
Copy link
Contributor

swirtSJW commented Oct 5, 2023

It is already pretty standard as the module offers this out of the box as an option for the "edit" version of the content.
image

@davidmpickett davidmpickett self-assigned this Nov 17, 2023
@davidmpickett davidmpickett removed their assignment Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Centralized Content A content type used across Facilties content Facilities Facilities products (VAMC, Vet Center, etc) Icebox Deprioritized for now Needs refining Issue status sitewide VAMC CMS managed product owned by Facilities team VBA for VBA facilities that are not Regional Offices Vet Center CMS managed product owned by Facilities team
Projects
None yet
Development

No branches or pull requests

4 participants