-
Notifications
You must be signed in to change notification settings - Fork 70
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
Comments
We omitted both of these fields from the FE design that was shown to Veterans for testing
We included them in the sketch design for continuity / just in-case we decide they are needed |
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. |
Historical context when we made a decision around this: #14123 (comment) |
@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! |
@xiongjaneg @mmiddaugh Now that we have the usability findings, I think we have enough information to make a decision here. Here is my proposal:
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.
Remove this field, usability testing showed it was not needed
Remove this field, usability testing showed it was not needed
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 |
@davidmpickett Would you please add to next UX Refinement to discuss? Thank you! |
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 |
@davidmpickett Is there anything in this ticket that needs to be reviewed at the CMS Team collab cycle meeting? |
@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 |
Spoke with @davidmpickett today to make sure I knew what was still outstanding on this ticket and was my responsibility. 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)
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. |
No additional feedback, closing |
Description
CMS Team reviewed #14891 and provided the following feedback:
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
Team
Please check the team(s) that will do this work.
The text was updated successfully, but these errors were encountered: