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

Update VBA fieldset of VA Service taxonomy for CMS Team feedback #15502

Closed
1 of 8 tasks
xiongjaneg opened this issue Oct 2, 2023 · 11 comments
Closed
1 of 8 tasks

Update VBA fieldset of VA Service taxonomy for CMS Team feedback #15502

xiongjaneg opened this issue Oct 2, 2023 · 11 comments
Assignees
Labels
Facilities Facilities products (VAMC, Vet Center, etc) Regional office CMS managed VBA product owned by the Facilities team sitewide UX VA services taxonomy CMS-managed product owned by the Facilities team

Comments

@xiongjaneg
Copy link
Contributor

xiongjaneg commented Oct 2, 2023

Description

CMS Team reviewed #14891 and provided the following feedback:

  • Should Type of Care be Type of Service since VBA services aren't necessarily health-related?
  • Should Patient Friendly Name be Service Name or something more general?
  • Should Common conditions be Keywords or something more general?
  • Also, will there be help text along with the field names to help editors?

This work is to incorporate their feedback into the fieldset.

Note for all new designs: Use USWDS v3 components when available. Preference is not to mix v3 and v1 components. Consult Amanda as needed.

Acceptance Criteria

  • Updated design in Figma based on CMS team feedback prior to and from design intent
  • Engineering review
  • A11y review
  • Shared with PO/PM for CMS Team review

Team

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

  • CMS Team
  • Public Websites
  • Facilities
  • Accelerated Publishing
@xiongjaneg xiongjaneg added Needs refining Issue status Facilities Facilities products (VAMC, Vet Center, etc) UX Regional office CMS managed VBA product owned by the Facilities team labels Oct 2, 2023
@davidmpickett
Copy link
Contributor

We omitted both of these fields from the FE design that was shown to Veterans for testing

  • Patient Friendly Name
  • Common conditions

We included them in the sketch design for continuity / just in-case we decide they are needed
https://www.sketch.com/s/891d33ae-152d-471c-ab5d-9aedf89cf6ff/p/578C3207-C5A3-4303-AB18-4AFC27AC54D2/canvas?annotation=477729a6-0bb3-4809-af59-acf41f32ec95

@davidmpickett
Copy link
Contributor

If the usability findings don't indicate that Veterans need either of these fields, then the path forward would be removing these fields, not renaming them.

@davidmpickett
Copy link
Contributor

Historical context when we made a decision around this: #14123 (comment)

@xiongjaneg
Copy link
Contributor Author

@davidmpickett @thejordanwood Let's leave those fields for now. We'll wait till end of sprint for Alexis' usability research results to see if they indicate a need for those placeholder fields or not. So at least one more kick down the road. Thank you!

@davidmpickett
Copy link
Contributor

davidmpickett commented Oct 17, 2023

@xiongjaneg @mmiddaugh Now that we have the usability findings, I think we have enough information to make a decision here. Here is my proposal:

  • Should Type of Care be Type of Service since VBA services aren't necessarily health-related?

Yes. We should also consult engineering on the trade offs of updating the machine name (currently field_vba_type_of_care). On the one hand, having the machine name formatted the same as the other type_of_care fields that do the exact same thing makes it easier to see the parallels in the code. On the other hand it is perpetuating legacy naming conventions that cause an equal amount of confusion for new devs.

  • Should Patient Friendly Name be Service Name or something more general?

Remove this field, usability testing showed it was not needed

  • Should Common conditions be Keywords or something more general?

Remove this field, usability testing showed it was not needed

  • Also, will there be help text along with the field names to help editors?

This form desperately needs some help text. It has not been prioritized previously because the number of editors who interact with it are so few and typically power users (e.g. Dave C, Randi H, Michelle M). It also doesn't have a KB article (but I made a ticket about that). This is likely what the bulk of the work on this ticket would be

@xiongjaneg
Copy link
Contributor Author

@davidmpickett Would you please add to next UX Refinement to discuss? Thank you!

@davidmpickett
Copy link
Contributor

Per UX refinement - proposal is approved. Path forward on machine name can be conversation between Drupal and FE on implementation. Drupal implementation can be follow up ticket.

This ticket can be updating Figma to remove/rename fields and add some help text

@xiongjaneg @jilladams ^ this ticket is candidate for Sprint 96

@xiongjaneg
Copy link
Contributor Author

@davidmpickett Is there anything in this ticket that needs to be reviewed at the CMS Team collab cycle meeting?

@davidmpickett
Copy link
Contributor

@xiongjaneg It might be worth acknowledging that we're already planning to address the feedback they gave us previously. The Figma may not be updated to reflect that before Wednesday's meeting. I already added this ticket to the Collab cycle epic so there's a record of it

@xiongjaneg xiongjaneg removed the Needs refining Issue status label Nov 7, 2023
@ALogsdon3
Copy link

ALogsdon3 commented Nov 14, 2023

Spoke with @davidmpickett today to make sure I knew what was still outstanding on this ticket and was my responsibility.
To summarize, here are the tasks left to do on this from the design side:

Update design in Figma based on pre-design intent CMS team feedback (There is a separate ticket with CMS design intent recommendations, but the scope of those recommendations goes beyond what's documented here and requires review with the team)

  • Update Type of care field to Type of service
  • Remove Patient friendly name field
  • Remove Common conditions field
  • Add help text under field labels where needed

I will update this comment as I complete each task.

Update 11/20/23: I made a pass at some help text. I added a couple of comments, too. One is on Service description because I wasn't able to add a help text field under the header. the other is on the URL help text because I had a question about KB articles related to URLs. @thejordanwood @davidmpickett have a look when you can and let me know what questions you have.

@xiongjaneg
Copy link
Contributor Author

No additional feedback, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Facilities Facilities products (VAMC, Vet Center, etc) Regional office CMS managed VBA product owned by the Facilities team sitewide UX VA services taxonomy CMS-managed product owned by the Facilities team
Projects
None yet
Development

No branches or pull requests

5 participants