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

sidebase V3 #80

Merged
merged 12 commits into from
Dec 10, 2022
Merged

sidebase V3 #80

merged 12 commits into from
Dec 10, 2022

Conversation

BracketJohn
Copy link
Contributor

@BracketJohn BracketJohn commented Dec 4, 2022

Closes #2
Closes #12
Closes #27

This PR:

  • switches from ant-design-vue to naive ui
  • switches from TypeORM to Prisma
  • removes the custom vite.conf.ts
  • adds a smaller, custom vitest.conf.ts
  • removes and reduces dev-related files:
    • removed: sidebase logos,
    • reduced: much of the readme docs,
    • reduced: example files to single welcome component
  • switches from antfu-eslint to nuxt-eslint rules
  • adds tailwind css vs code as a recommended extension
  • expands testing:
    • E2E app example using nuxt-test-utils
    • unit-test-example using nuxt-test-utils

ToDos:

],
"tailwindCSS.experimental.configFile": ".nuxt/tailwind.config.cjs",
"files.associations": {
"*.ccs": "tailwindcss"

Choose a reason for hiding this comment

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

This is meant to be *.css I think!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What do you mean? This is to improve editor-support and taken from https://tailwindcss.nuxt.dev/tailwind/editor-support

Choose a reason for hiding this comment

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

Editor support would be improved if the association is to CSS files - it's just misspelt here 🙂

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah of course - very good catch :D Fixed (:

@BracketJohn BracketJohn merged commit c96e65e into main Dec 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants