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

Fix broken Teleterm stories #22657

Merged
merged 2 commits into from
Mar 6, 2023
Merged

Conversation

gzdunek
Copy link
Contributor

@gzdunek gzdunek commented Mar 6, 2023

This PR has switched Teleterm's ThemeProvider to use the named export. Unfortunately, I missed the storybook config, so it stopped working.

@ravicious
Copy link
Member

We could probably refactor Storybook files to use TypeScript instead to avoid similar mistakes in the future.

https://stackoverflow.com/a/69040674/742872

@ravicious
Copy link
Member

However, the caveat here is that you cannot use {"target": "ESNext"} in your typical tsconfig.json because node doesn't support ES module natively, yet.

😖

@gzdunek
Copy link
Contributor Author

gzdunek commented Mar 6, 2023

We could probably refactor Storybook files to use TypeScript

I just tried it and it seems to work, I will play with later and create a separate PR.

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from avatus March 6, 2023 11:28
@gzdunek gzdunek added this pull request to the merge queue Mar 6, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 6, 2023
@gzdunek gzdunek added this pull request to the merge queue Mar 6, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 6, 2023
@gzdunek gzdunek enabled auto-merge March 6, 2023 14:54
@gzdunek gzdunek added this pull request to the merge queue Mar 6, 2023
Merged via the queue into master with commit 793210e Mar 6, 2023
@public-teleport-github-review-bot

@gzdunek See the table below for backport results.

Branch Result
branch/v12 Create PR

@gzdunek gzdunek deleted the gzdunek/fix-broken-teleterm-stories branch March 20, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants