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

with-stitches example fixes #28650 #28652

Merged
merged 12 commits into from
Sep 7, 2021

Conversation

RobEasthope
Copy link
Contributor

The with-stitches example is broken presumably due to changes in the Stitches API now that it's hit 1.0.0. I've created an issue already: #28650

I've made three small changes:

  • Replaced the createCss function in stitcheds.config.js with createStitches
  • Replaced the getCssString function with getCssText in stitches.config.js and pages/_documents.jsx
  • Bumped @stitches/react to ^1.0.0 🎉

I've left the rest of the example code alone as I'm still exploring the library after seeing yesterdays release annoucement.

@peduarte Is there anything else that needs to updated?

@ijjk ijjk added the examples Issue/PR related to examples label Aug 31, 2021
@peduarte
Copy link

peduarte commented Sep 6, 2021

Hey, thanks for this!

The utils API has changed also. So that'll need to be updated in the config.

@RobEasthope
Copy link
Contributor Author

Hey, thanks for this!

The utils API has changed also. So that'll need to be updated in the config.

No worries, I've updated the utils syntax as well

Copy link

@peduarte peduarte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. LGTM! 🚀

Copy link
Member

@leerob leerob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@kodiakhq kodiakhq bot merged commit e0b3849 into vercel:canary Sep 7, 2021
bordeo pushed a commit to filoblu/next.js that referenced this pull request Sep 7, 2021
* canary:
  Remove unused profiler (vercel#28844)
  Remove outdated webpack conformance experiment (vercel#28846)
  v11.1.3-canary.13
  v11.1.3-canary.12
  Upgrade upload-artifact action
  v11.1.3-canary.11
  v11.1.3-canary.10
  Update precompiled
  with-stitches example fixes vercel#28650 (vercel#28652)
  (docs) Add `permanent` property in redirect object (vercel#28852)
  (docs) Remove duplicate comma (vercel#28850)
  v11.1.3-canary.9
@vercel vercel locked as resolved and limited conversation to collaborators Jan 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue/PR related to examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants