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

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
282 stars 203 forks source link

Add Descriptive Tags for Design System Templates #78505

Closed erinrwhite closed 7 months ago

erinrwhite commented 7 months ago

User Story

As a Governance team member, I want to ensure all tags and references for DS Templates are included in the DraftingTemplate so that Staging Review issues tickets are published with valid references for VFS teams.

Assignee: @erinrwhite Peer Reviewer: @shiragoodman

Description

In a previous ticket #76875, we made sure that the Drafting Template contained references to all Design System components and patterns. We did not include Templates at the time, but will do so now.

When we add Templates, they should be added alongside the Design System Patterns and we should change the title in the DraftingTemplate to Design System Patterns/Templates.

In addition:

Impacted Artifacts

Tasks

Peer Review

To be completed by peer reviewer

Acceptance Criteria

Team Notification

How to prepare this issue

Refinement

erinrwhite commented 7 months ago

@shiragoodman I have added the new descriptive tags to the source of truth, and to the Drafting Templates. Both of the drafting templates' logic in the Design System Pattern or Template column needs to be updated to reflect the new rows. If you give me access to the protected ranges in the sheet, I can make an attempt at it! or I can leave this change to you.

shiragoodman commented 7 months ago

I can give you access. I have been considering removing all protections within the sheet anyways since it seems to be more problematic than it is helpful. I'll let you know once it's ready for you.

erinrwhite commented 7 months ago

@shiragoodman

Source of truth

Drafting template updates for your review

shiragoodman commented 7 months ago

Thanks @erinrwhite!

As for the ticket itself, I did add a line under Description stating that this ticket will also address adding a tag for imposter components. Thank you for including that in your work.

Source of truth looks good. I like that you've included the "Other tag" option to list out launch-blocking and imposter component. We should also add the Design System Review label per ticket #75629. Since we will always add both the Design System team label (platform-design-system-team) and the review label (design-system-review), Allison set up the DraftingTemplate to print out "platform-design-system-team,design-system-review" in the labels column of the Github Upload tab. I am not sure how you'd like this reflected in the source of truth.

I'm seeing a problem with the pattern/template label and pattern/template reference link columns (columns X and Y) in the Findings tab not populating correctly. I know why the problem is happening, but I think it might be best to sync with you to explain the problem and we can figure out a solution together.. since there's a few different ways we can solve this.

Let me know if you have time today to meet!

erinrwhite commented 7 months ago

@shiragoodman Thank you! booking us a time at 1:15!

erinrwhite commented 7 months ago

@shiragoodman thanks for the helpful meeting today! I think this new formula will fix things up for us.

For your review: Drafting Template draft version Drafting Template - DST draft version

Updated Source of Truth πŸ‘πŸ»

erinrwhite commented 7 months ago

Moved back to in-progress - per discussion on Slack we will be adding in Design System Foundation to Source of Truth and Drafting Templates as part of this ticket.

erinrwhite commented 7 months ago

@shiragoodman ready for your review AGAIN! πŸ˜‚

Updated Source of Truth

Drafting Template draft version Drafting Template - DST draft version

erinrwhite commented 7 months ago

Also, can we bump this ticket up to a 3? πŸ˜…

shiragoodman commented 7 months ago

thanks @erinrwhite . Yes, bump it to a 3. We've added on quite a bit.

I will review your updates in the morning!

shiragoodman commented 7 months ago

@erinrwhite awesome! Everything is working as expected. The only comment I have is to ask you to remove the permissions since this is now a document you own. It might also be a good idea to create a copy of the baseline DT for both VFS and DST in case someone accidentally overwrites one of the formulas.

We should be good to close this out once you make the changes.

erinrwhite commented 7 months ago

@shiragoodman Thank you! I had changed the protections from "Restrict who can edit this range" to "Show a warning when editing this range". What do you think about the keeping the permissions as "Show a warning"? It would still show a warning if someone was trying to edit something, but would not restrict who can edit. I think it could be a handy protective measure but still allow everyone to edit if they actually intend to.

I like the idea of a backup - will do that. Let me know what you think about keeping the warnings.

shiragoodman commented 7 months ago

great idea @erinrwhite ! The warnings would be very helpful. Let's do that!

erinrwhite commented 7 months ago

Good deal.

Created Backups of drafting templates

And updated drafting templates are now renamed and live Drafting Template Drafting Template - DST

Closing!