department-of-veterans-affairs / va.gov-cms

Editor-centered management for Veteran-centered content.
https://prod.cms.va.gov
GNU General Public License v2.0
98 stars 69 forks source link

Front end: Vet Center unpublished services should not be showing on VA.gov #16362

Closed xiongjaneg closed 2 months ago

xiongjaneg commented 10 months ago

User Story or Problem Statement

As a Veteran, I want to know which services are available at my Vet Center and should not see services that aren't available.

As a Vet Center editor, I want to show only services that are available at my Vet Center and my Vet Center page shouldn't show services that aren't available that I've unpublished.

Per Slack thread, unpublished Vet Center services should not be showing on VA.gov.

Per Steve in Slack, The Vet Center services query is not filtering for Published (status) image

Sequencing for launch

August 15 change management will unblock Drupal #18003 to merge the Drupal PR. This ticket should follow #18003 merge and use the Graph QL from that branch / PR.

Acceptance Criteria

davidmpickett commented 10 months ago

IMO there are a few different phases here:

mmiddaugh commented 10 months ago

Re: option to add a View Block of services to the Dashboard which would direct editors to admin/content filtered properly for services and their vet section

Notable differences in editorial experience

image

admin/content service edit view ![image](https://github.com/department-of-veterans-affairs/va.gov-cms/assets/55411834/a2195dc2-eaa6-4d49-bbe0-9dcbcc56b4bb)
mmiddaugh commented 10 months ago

If we get into Dashboard changes related to this, #7269 Improve Vet Center editor experience - dashboard and other may be relevant

davidmpickett commented 10 months ago

Historical ticket noting the ambiguity of Vet Center service moderation state: https://github.com/department-of-veterans-affairs/va.gov-cms/issues/7144

davidmpickett commented 10 months ago

Re: option to add a View Block of services to the Dashboard which would direct editors to admin/content filtered properly for services and their vet section

Notable differences in editorial experience

  • current admin/content workflow does not distinguish required services (which cannot be removed by editor) from optional services (which editors can remove)

It might be important to note that the only thing currently preventing Vet Center editors from Archiving Required Services is the fact that they are unlikely to ever go to admin/content. Required services aren't actually structurally different than non-required services in a content moderation workflow sense.

On Staging as a Vet Center editor, I was able to Archive a required service in my section

screencapture-staging-cms-va-gov-st-louis-vet-center-service-st-louis-vet-center-addiction-and-substance-abuse-care-2023-12-11-19_15_34

Although give that unpublished services are currently showing up on VA.gov, even archiving a service wouldn't necessarily remove it. 🤯

xiongjaneg commented 9 months ago

@xiongjaneg There is an accompanying Drupal ticket. This is not blocked by Drupal.

When we do this on front end, there will be change management with editors who have what they thought were published services but with this fix, these services will not be published.

xiongjaneg commented 9 months ago

This fix may also have the advantage of making it apparent to editors when they've only saved in Draft and never Published, because when they check their published page, draft content won't show.

xiongjaneg commented 9 months ago

Some of these issues may be impacted by the inline editor ticket #15125. Should this wait? Likely no, there is risk in publishing content not intended for publishing.

mmiddaugh commented 8 months ago

@xiongjaneg depending on level of effort, Vet Center service editorial UX does not show national description #17177 may need to ride along with this work

xiongjaneg commented 8 months ago

blocked by IEF removal ticket

swirtSJW commented 8 months ago

This is blocked because currently if an editor edits a VetCenter Service, due to IEF, they do not have the option to publish their change. So if we stop showing draft content, they have no way to get new edits out to the FE.

jilladams commented 2 months ago

This change would mean that the 30 draft services that currently appear in the front-end need to be published in order to continue appearing on the FE.

We need to do CM for editors before we make this change.

Agile6MSkinner commented 2 months ago

@mmiddaugh is working with VetCenters with draft services to get them addressed. @mmiddaugh should we consider this one unblocked per our conversation?

mmiddaugh commented 2 months ago

@eselkin For validation purposes, here are a few Vet Center services in draft as of this morning

I did not see any of these services displayed on the relevant pages in Prod

jilladams commented 2 months ago

That's a verification in prod! Closing.