-
Notifications
You must be signed in to change notification settings - Fork 7
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
feat(icon): update "new", "active", "inactive", and "status" icons #986
Conversation
This pull request has been linked to Shortcut Story #213825: Flag status UI updates. |
🦋 Changeset detectedLatest commit: 0ecb4a6 The changes in this PR will be included in the next version bump. This PR includes changesets to release 24 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Size Change: 0 B Total Size: 171 kB ℹ️ View Unchanged
|
.changeset/ninety-ravens-argue.md
Outdated
@@ -0,0 +1,5 @@ | |||
--- | |||
'@launchpad-ui/icons': minor |
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.
'@launchpad-ui/icons': minor | |
'@launchpad-ui/icons': patch |
Non-breaking 0.x
versioning changes
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 lockfile changes seem undesired. What version of pnpm are you using?
I was using an earlier version at first but I then upgraded to 8.7.1. I was on v7 but I had some issues with it |
Could you revert the changes to the lockfile? |
Absolutely! I think I found how I caused that. I just reverted those changes now. |
Summary
Screenshots (if appropriate):
Testing approaches
Locally running unit, component, and SSR tests.