-
Notifications
You must be signed in to change notification settings - Fork 212
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(clerk-js): Add dev mode notice to components #3511
base: main
Are you sure you want to change the base?
feat(clerk-js): Add dev mode notice to components #3511
Conversation
🦋 Changeset detectedLatest commit: b5a6868 The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 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 |
!preview |
d0c8cd0
to
2cbdac6
Compare
Hey @octoper, your preview is available.
|
2cbdac6
to
8e9316d
Compare
!preview |
@desiprisg can we have a screenshot of how this looks like? |
Adding these! Feel free to check the preview deployment as well! |
a6e46ac
to
8f84fb9
Compare
width: 'fit-content', | ||
paddingLeft: theme.space.$3, |
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.
❓ Wouldn't the removal of the padding cause issues when the devmode notice is not shown?
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.
If I remember correctly, we are changing how this appears completely. We want it centered now.
{shouldShowTagOrLinks && ( | ||
<Card.ClerkAndPagesTag | ||
withFooterPages | ||
sx={t => ({ padding: `${t.space.$4} ${t.space.$8}` })} |
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.
Same as above
Hey @desiprisg , whats the status of this one? |
We were waiting for user button / org switcher designs before merging this one. |
8f84fb9
to
5052798
Compare
!preview |
5052798
to
17d5afa
Compare
@octoper hey :) is this good to go? if not, do you need help with anything at this point? |
There were no designs for |
17d5afa
to
9d79ddb
Compare
!preview |
05a4721
to
e4f6058
Compare
!preview |
@octoper do we offer the ability for a developer to turn this off in Dev? |
e4f6058
to
1c8b152
Compare
1c8b152
to
b5a6868
Compare
Description
Add a development mode notice to our components to avoid going to productions with dev keys by accident.
Checklist
npm test
runs as expected.npm run build
runs as expected.Type of change