Skip to content
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

USWDS-Site: Instructions for migrating from USWDS 1.x #1739

Merged
merged 6 commits into from
Aug 29, 2022
Merged

Conversation

amyleadem
Copy link
Contributor

@amyleadem amyleadem commented Aug 8, 2022

Summary

Added clarity for users migrating from USWDS 1 to USWDS. We included a note at the top of the 3.0 Migration guide explaining that users updating from 1.x should first update to version 2 via the 2.0 Migration guide.

Related issue

Closes #1733

Related discussion: uswds/uswds#4895

Preview link

Preview link: Migrating to 3.0

Problem statement

Users should be able to easily find all the needed instructions for upgrading any version of USWDS to version 3. However, users who are upgrading from 1.x do not have clear instructions for all the steps they need to take to get to 3.x.

Solution

Providing instructions to 1.x users that they must first follow instructions in the 2.0 migration will help clarify the path to get to version 3.

Possible additional solution not included in this PR: Add an additional note in Section 4 of the 3.0 migration guide. I left this out because the initial note clarifies that this is only for migrations from 2 to 3, and additional instructions felt unnecessary. Please let me know if you believe this would be a helpful addition.

Testing and review

To review:

  1. Confirm that the instructions provided are clear and comprehensive enough that 1.x users can successfully upgrade to USWDS 3.
  2. Proof the copy for any errors

Before opening this PR, make sure you’ve done whichever of these applies to you:

  • Run git pull origin [base branch] to pull in the most recent updates from your base and check for merge conflicts. (Often, the base branch is develop).
  • Run npm test and confirm that all tests pass.

@amyleadem amyleadem marked this pull request as ready for review August 11, 2022 16:05
Copy link
Contributor

@bonnieAcameron bonnieAcameron left a comment

Choose a reason for hiding this comment

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

Hi Amy! This looks great. Can we add a small bit about how someone can tell what version they are on? So it could be like:

Note: This guide provides instructions for upgrading from USWDS 2 to USWDS 3. If your project uses USWDS 1, you will need to update according to the USWDS 2.0 migration guide before proceeding with these steps. Not sure what version you're on? Here's how to check.

@amyleadem
Copy link
Contributor Author

amyleadem commented Aug 11, 2022

@bonnieAcameron Great suggestion! I added this copy in commit a0b0f6c

Copy link
Contributor

@bonnieAcameron bonnieAcameron left a comment

Choose a reason for hiding this comment

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

Looks great! Checked for typos and clarity.

Copy link
Contributor

@mejiaj mejiaj 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!

I was able to verify the information is correct and all links work.

@mejiaj mejiaj requested a review from thisisdano August 18, 2022 19:47
@thisisdano thisisdano merged commit 53ea49f into main Aug 29, 2022
@Elvis-haley
Copy link

I am Justice Amy Coney Barrett. Elvis Haley is a Navy Zero.

@amyleadem amyleadem deleted the al-migration-1x branch July 17, 2023 14:40
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.

Update 3.0 Migration guide to include instructions for 1.x
5 participants