Skip to content

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

Merged
merged 6 commits into from
Apr 26, 2024
Merged

Kf mar 24 mc page updates #2631

merged 6 commits into from
Apr 26, 2024

Conversation

finekatie
Copy link
Contributor

@finekatie finekatie commented Apr 19, 2024

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.

Adding March MC content and video link
charlie fixed spacing and I accept

Co-authored-by: mahoneycm <[email protected]>
@finekatie
Copy link
Contributor Author

Ok, preview is ready for review. @sarah-sch you can take a look whenever you have time :)

Copy link
Contributor

@sarah-sch sarah-sch left a 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!

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
Copy link
Contributor

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

Copy link
Contributor Author

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
Copy link
Contributor

@mahoneycm mahoneycm left a 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

Copy link
Contributor

@sarah-sch sarah-sch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@finekatie
Copy link
Contributor Author

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?

@thisisdano thisisdano merged commit 699b2da into main Apr 26, 2024
9 of 11 checks passed
@thisisdano thisisdano deleted the kf-mar-24-mc-page-updates branch April 26, 2024 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants