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 storybook DX by removing default type check config #4699

Merged
merged 1 commit into from
Mar 8, 2022

Conversation

thedavidprice
Copy link
Contributor

Part one of two resolving #4676 by removing TS config added here

This is a change to fix a DX problem with new Redwood TS projects and/or cases where Cells design in Storybook is broken because query types do not exist.

As a next step, we'll update the RW Storybook doc to explain how to enable type check. We'll also need to cover the caveat about Cells, which needs discussion.

@thedavidprice thedavidprice added the release:fix This PR is a fix label Mar 8, 2022
@thedavidprice thedavidprice merged commit 70f4eeb into main Mar 8, 2022
@thedavidprice thedavidprice deleted the dsp-remove-storybook-ts-check branch March 8, 2022 23:37
@jtoar jtoar added this to the next-release milestone Mar 8, 2022
thedavidprice added a commit that referenced this pull request Mar 8, 2022
thedavidprice added a commit that referenced this pull request Mar 9, 2022
dac09 added a commit to dac09/redwood that referenced this pull request Mar 11, 2022
…d into feat/auth-checks-smoke-test

* 'feat/auth-checks-smoke-test' of github.com:dac09/redwood: (21 commits)
  Remove supertokens-node from packages/api dependencies (redwoodjs#4715)
  fix(auth): hasRole handles when currentUser.roles is a string (redwoodjs#4678)
  Update dependency systeminformation to v5.11.7 (redwoodjs#4716)
  Update dependency webpack-manifest-plugin to v5 (redwoodjs#4693)
  Update graphqlcodegenerator monorepo (redwoodjs#4714)
  Update dependency @clerk/types to v1.28.3 (redwoodjs#4708)
  Update dependency @testing-library/react to v12.1.4 (redwoodjs#4709)
  Update dependency pino to v7.8.1 (redwoodjs#4703)
  Update dependency fastify to v3.27.4 (redwoodjs#4702)
  Update dependency @clerk/clerk-sdk-node to v2.9.8 (redwoodjs#4707)
  Update dependency @types/react to v17.0.40 (redwoodjs#4711)
  Update dependency @clerk/clerk-js to v2.17.3 (redwoodjs#4706)
  Fix react/prop-types lint warnings (redwoodjs#4674)
  Allow the number 0 for numericality validation values (redwoodjs#4700)
  update yarn.lock
  v0.49.1
  update yarn.lock
  remove storybook type check (redwoodjs#4699)
  add bin proxy for rw-log-formatter to core (redwoodjs#4695)
  remove storybook type check (redwoodjs#4699)
  ...
@noire-munich
Copy link
Collaborator

Thank you for this!
It cut us off Storybook for months, we'll be able to get back to it now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:fix This PR is a fix
Projects
No open projects
Status: Archived
Development

Successfully merging this pull request may close these issues.

None yet

3 participants