-
Notifications
You must be signed in to change notification settings - Fork 201
docs: adds tag for migrated components #3688
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
docs: adds tag for migrated components #3688
Conversation
|
File metricsSummaryTotal size: 1.38 MB* 🎉 No changes detected in any packages * Size is the sum of all main files for packages in the library.* An ASCII character in UTF-8 is 8 bits or 1 byte. |
🚀 Deployed on https://pr-3688--spectrum-css.netlify.app |
@@ -48,7 +48,6 @@ export default { | |||
packageJson, | |||
metadata, | |||
}, | |||
tags: ["migrated"], |
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.
Meter isn't quite migrated. We are missing a new S2 size.
0f3e43e
to
b3a5550
Compare
Description
Some of our migrated components from 2024 (like textfield, search, switch, button, etc), had the migrated tag added to the components' metadata. You could find
tags: ["migrated"]
in the story files. However, at some point we stopped adding that tag, and because we've gotten back to migrated S2 components, it would probably be good to add this tag to all of our currently migrated components.This work adds that migrated tag to the components that are missing it as of 5 May 2025.
Jira
CSS-1191 (ticket for this work)
CSS-604 (epic for S2 Migration)
How and where has this been tested?
Please tag yourself on the tests you've marked complete to confirm the tests have been run by someone other than the author.
Validation steps
tags: ["migrated"]
in the repo. You should get 45 components returned 🥳Regression testing
Validate:
Screenshots
To-do list