-
Notifications
You must be signed in to change notification settings - Fork 194
Kf mar 24 mc page updates #2631
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
Conversation
Adding March MC content and video link
charlie fixed spacing and I accept Co-authored-by: mahoneycm <[email protected]>
Ok, preview is ready for review. @sarah-sch you can take a look whenever you have time :) |

There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The video isn't working, and I left some in-line notes about updating the ID. The rest of the content looks good!
_data/monthly-calls.yml
Outdated
description: | | ||
Our USWDS team did a great live [tutorial on customizing and extending USWDS](https://github.com/uswds/uswds-tutorial). We covered working with settings, applying tokens, handling style conflicts, and ensuring good color contrast. | ||
date: March 2024 | ||
id: M2SVDceCM |

There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The embedded video isn't playing for me. I'm wondering if we need these underscores in the ID: M2_SVDc_eCM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we do. I thought they were glitches in a version of the link I saw. I'll add 'em back. I didn't have a chance to preview any of this before I sent to you for review. Busy day!
Fixed video link
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Notes
Looks good to me!
Testing checklist
- Visual display of content is consistent with previous sections
- Links work as expected
- Video works as expected
- Content looks good and is easy to read
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Hey @thisisdano, I got psyched to see this approved and am noticing some checks failed here too. Should I be bothering you to check or should I be asking Charlie? |
Adding the March event, along with a description and related links, to the USWDS monthly call page.
Summary
This purpose of this PR is to update the monthly call page to include a video (and supplemental information) for the March 2024 monthly call. This update will keep the page relevant and timely.
Important
This PR is based off the changes from #2576. We should wait to merge until that is merged into main.
Related issue
#2602
Preview link
Monthly calls page
Problem statement
The March 2024 event needs to be added to the USWDS monthly calls page (and accompany the Feb monthly call content also going on to the page)
Solution
This action will add the March 2024 event to the monthly call page so that the USWDS community can enjoy it!
Testing and review
Please proof the new content and make sure it's formatted correctly.