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

feat: update stack and /open-design #148

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

feat: update stack and /open-design #148

wants to merge 9 commits into from

Conversation

replygirl
Copy link
Contributor

@replygirl replygirl commented Jun 20, 2023

  • migrate from uno to windicss
  • update deps
  • enforce node 18 and yarn 3.2
  • change /open-design archive design
  • add discord + miro ctas on both pages
  • remove notion api

NOTE: unocss isnt playing nicely with nextjs all the time. the fix is to save next.config.js to force a server restart without killing the process. idk why this happens. there is an unocss engine using postcss instead of a webpack plugin which solves this problem, but it's not stable yet

BREAKING: slight differences with uno syntax. https://unocss.dev/presets/wind#differences-from-windi-css

@vercel
Copy link

vercel bot commented Jun 20, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dual-power-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 7, 2023 4:22am

@replygirl
Copy link
Contributor Author

replygirl commented Jun 20, 2023

📝 Changed routes:

/ /open-design
Screenshot of /
(view full size)
Screenshot of /open-design
(view full size)

And 2 other routes:

4 deleted routes:

  • /api/_utils/create-notion-getter
  • /api/docs
  • /api/events
  • /api/status

Commit f42271e (https://dual-power-2kolcjgol-blacksocialists.vercel.app).

uno.config.ts Outdated Show resolved Hide resolved
@feralislatr
Copy link
Collaborator

i see that the font files were deleted from public/fonts, was this intentional? those were being used in storybook; not sure if we should just move them somewhere else.

going to see if i can get the styles to show up in storybook otherwise

@replygirl
Copy link
Contributor Author

i see that the font files were deleted from public/fonts, was this intentional? those were being used in storybook; not sure if we should just move them somewhere else.

going to see if i can get the styles to show up in storybook otherwise

uno loads them. if you can't get uno to load them for storybook you could use css imports from google fonts in main.js

@feralislatr
Copy link
Collaborator

feralislatr commented Jul 2, 2023

so i got storybook working with unocss, but i have to use node 16 because we're using webpack 4. is it worth having the .nvmrc specify node 18?

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.

2 participants