Visitar URL original
Add changelog items for USWDS 3.5.0 by thisisdano · Pull Request #2096 · uswds/uswds-site · GitHub
Skip to content

Add changelog items for USWDS 3.5.0 - #2096

Merged
thisisdano merged 101 commits into
release-3.5.0from
dw-changelog-button-outline-whc
Jun 9, 2023
Merged

thisisdano merged 101 commits into
release-3.5.0from
dw-changelog-button-outline-whc

Conversation

@thisisdano

@thisisdano thisisdano commented May 8, 2023 •

Copy link
Copy Markdown
Contributor

This collects changelog items for any meaningful change in USWDS 3.5.0:

Warning
This will fail HTMLProofer until we publish USWDS 3.5.0 in GitHub.

Note
This PR is a draft. I will keep updating it as we prepare USWDS 3.5.0.

Note
All dates will appear as NNNN-NN-NN until we have a solid date for this release.


Related PR

USWDS: USWDS 3.5.0 #5323

@mejiaj

mejiaj commented Jun 7, 2023

Copy link
Copy Markdown
Contributor

@thisisdano @amyleadem removed banner accordion init changelog item in ab0805f.

@amyleadem amyleadem left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@thisisdano Took another look at the changelogs. Found just a couple of small things:

  • Confirm that all PRs in the release list are accounted for
  • Confirm all release numbers are accurate
  • Confirm all dates follow NNNN-NN-NN format
    • [ ] One item needs to be updated, as identified in comment below We can disregard this because this changelog item has already been added to main
  • Confirm listed repo is accurate

Notes

  • Discovered a couple of 404 items in the PR build errors. Opened a PR to fix:
    • Merge 404 fixes in #2128
  • I also am seeing errors in test_a11y_mobile. Initial tests seem to suggest that it is isolated to this PR. I can look into it more tomorrow.
    • Error message: Ensure that scrollable region has keyboard access
    • Investigate and resolve a11y errors

Comment thread package.json Resolved
James Mejia added 2 commits June 8, 2023 09:03
@mejiaj

mejiaj commented Jun 8, 2023

Copy link
Copy Markdown
Contributor

uswds/uswds#5311
Header: Fixed a bug that caused inaccurate megamenu template markup. The markup in megamenu code examples now accurately includes the usa-megamenu class.
Preview: Header changelog →

This note doesn't currently include a changelog entry. From cloud pages preview:
image

@thisisdano
thisisdano marked this pull request as ready for review June 9, 2023 03:51
@thisisdano

Copy link
Copy Markdown
Contributor Author

OK, I think this is ready for a final review pass before its merge into the release branch

@amyleadem

Copy link
Copy Markdown
Contributor

All updates are looking good so far.

HTML Proofer found some new 404s on the website policies page, but I didn’t feel 100% confident about their replacements and it felt particularly important to get these right. Here are the broken links with my best guesses for possible replacements:

@thisisdano, @mejiaj, @mahoneycm Any thoughts?

@thisisdano

Copy link
Copy Markdown
Contributor Author

Yes to Budget and performance reports and Orders and directives. Confirmed with Wayback Machine.

I also can't locate Open government initiatives. We will have to leave it broken for now until I can get some better information

@thisisdano
thisisdano merged commit 6b787af into release-3.5.0 Jun 9, 2023
@thisisdano
thisisdano deleted the dw-changelog-button-outline-whc branch June 9, 2023 20:57
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