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

Accidental unmerged PR #67

Conversation

tmeasday
Copy link
Member

@tmeasday tmeasday commented Sep 1, 2023

📦 Published PR as canary version: 0.0.50--canary.67.49b39b0.0

✨ Test out this PR locally via:

npm install @chromaui/[email protected]
# or 
yarn add @chromaui/[email protected]

@linear
Copy link

linear bot commented Sep 1, 2023

AP-3485 Display the project token and project id to the user in case of failure - enable user to configure manually.

When a user selects a project from the project picker, we automatically add these to the main.js file.

However, in the event that the user has a very custom configuration, there is a possibility that adding it automatically will fail. This is a suggestion that we display the project token and project id that the user selects, before trying to automatically add it to the storybook configuration.

In this case, it may also make sense to provide a prompt to the user before trying to automatically modify their configuration. Perhaps a secondary button that skips automatic configuration (and lets the user know how to manually configure it). Otherwise, we would always attempt to configure it and potentially break their configuration.

Even in the event of a failure, the projectToken and projectId are stored in localstorage on the client and an environment variable on the server to not require a storybook restart, so the addon allows the user to continue without finishing the configuration step. This may not be an issue, but if we allow user to manually update, then this may lead to some confusion if they skip manually and automatically updating it.

@tmeasday tmeasday merged commit b72a8ad into main Sep 1, 2023
@tmeasday tmeasday deleted the tom/ap-3485-display-the-project-token-and-project-id-to-the-user-in-case branch September 1, 2023 03:31
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.

1 participant