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
99 stars 69 forks source link

CMS: Update how COVID status feature works on VAMC facilities #10708

Closed dsinla closed 1 year ago

dsinla commented 2 years ago

Description

The current (Sep 2022) bulleted descriptive text for COVID status policies does not meet the operational reality at the facilities - it's not granular enough to give Veterans the variation of policies that are in place at each facility.

The content team is providing guidance for editors, which will need to be added to a new KB article that will be referenced from the COVID-19 status area of the editorial interface.

User Stories

As a Veteran, I want information about COVID policies that reflects the reality at the facility, in order to prepare for my visit. As a VAMC editor, I want guidance on the best way to author content that accounts for the reality of the policies at my facility.

Acceptance Criteria

CMS Team

Please check the team(s) that will do this work.

swirtSJW commented 2 years ago

I updated the Guidelines related AC

dsinla commented 2 years ago

Is this a good first ticket for Blake? Need to weigh in among priorities - consult with Erika

omahane commented 1 year ago

@davidmpickett Do we have locked down language and design for this issue? I see areas of ambiguity here:

davidmpickett commented 1 year ago

Tagging in @BlakeOrgan here as he was involved in these conversations more directly.

omahane commented 1 year ago

@davidmpickett @BlakeOrgan Does this only apply to VAMC facilities? I know we only have COVID status information on those, as opposed to Vet Centers and VBA facilities, but I want to make sure I understand the scope of the ticket.

omahane commented 1 year ago
davidmpickett commented 1 year ago

Jay noted that there will likely be a lighthouse implication

JQuaggles commented 1 year ago

@omahane Is this refined enough to do? And is 5 still accurate?

omahane commented 1 year ago

Here's the most recent design that @BlakeOrgan provided: https://www.figma.com/file/IYpeP2ZAmBTT8jqhdtJklu/Covid-%2F-Facility-Status?node-id=128%3A32180&t=GcTmUSaSHKT2y5BN-0

davidmpickett commented 1 year ago

Per conversation in Refinement last week, the parts about "Normal services and hours" being unselectable are no longer the desired direction and I removed the ACs about them

davidmpickett commented 1 year ago

Notes from a call with @omahane @swirtSJW

Facility Supplemental Status Taxonomy Terms

VAMC System Operating Status Content Type

VAMC Facility Content Type

Front End

Timing

swirtSJW commented 1 year ago

I have scoped the ACs on this ticket to cover just the facility node content type. If it is determined later that we will also alter the inline entity forms on the locations page then it should be a separate ticket.

omahane commented 1 year ago

@JQuaggles Can you help us address the implications for the System Operating Status edit pages?

JayDarnell commented 1 year ago

Adding myself so I can look at this while @omahane figures out his weird 502 bad gateway error

BlakeOrgan commented 1 year ago

I would also like to point out that since the Facilities team is making this update, any associated KB articles will also need to be updated to reflect this new way of doing things. It doesn't look like that is part of this ticket's AC's, but I'm also not finding an associated ticket where that work would be done.

JayDarnell commented 1 year ago

@BlakeOrgan @Dottisea I created an issue 12360 above

omahane commented 1 year ago

@BlakeOrgan In talking about the necessary front-end ticket to accompany this work, I realized that we may need to change the design.

https://user-images.githubusercontent.com/766573/217942817-33fb719d-d842-4457-9ba7-91eb70ee16d8.mov

We have 2 distinct fields for the status:

The Status name is standardized and gets a style distinct from the description on the front-end. The Status description is now customizable, so it might include the boilerplate language from the term description or it may not.

Screen Shot 2023-02-09 at 3 26 19 PM

If we concatenate them together in the WYSIWYG, there will be no way for the front-end to know for certain which content is the name and which is the description, especially as the editors may change any content in the WYSIWYG, removing the standardized language of the Covid status name. Therefore, I'm recommending that the WYSIWYG not include the Covid status name as a part of the content that is populated by choosing one of the statuses. Instead, only the Covid status description from the term would populate the WYSIWYG.

Screen Shot 2023-02-09 at 3 58 28 PM

On node:view, however, the content would show as it will appear on the front-end. Screen Shot 2023-02-09 at 4 05 30 PM

BlakeOrgan commented 1 year ago

@omahane I updated the Figma design to remove the covid status name from the text input, however the default text that is being put in that text box is coming from a vocabulary so it will need to be updated here if it hasn't already to reflect the new default text that gets pulled in based on the selected covid level. @swirtSJW Can correct me if I'm misspeaking here.

@davidmpickett also made a comment on Dec 23 that might be relevant to this. It might be worth double checking to see if the info that they want to auto-populate is still accurate.

Facility Supplemental Status Taxonomy Terms No code changes needed here PM to confirm with VHADM that the current descriptions for Low, Medium, & High are what we want to auto-populate in the fieldset for editors. Likely there's no change. This is not a blocker for the dev work, just want to make sure we're 100% clear on what the default text should be for the 3 levels.

omahane commented 1 year ago

Thanks, @BlakeOrgan. I appreciate your flexibility. Regarding the text, I'll confirm what needs to be live when we roll this out with @davidmpickett (CC: @Dottisea )

12111 Captures this requirement.