magicui icon indicating copy to clipboard operation
magicui copied to clipboard

Bug fixes: Fixes #78 & #81

Open BankkRoll opened this issue 1 year ago • 6 comments

Changed

  • Updated PNPM version requirement in package.json from >=9 to >=8 to support a wider range of PNPM versions, while still recommending PNPM version 9.0.4. Fixes #78

Added

  • Added .gitattributes file to ensure consistent line endings across different operating systems. This should help avoid issues related to line ending differences when working on various os. Fixes #81

BankkRoll avatar May 27 '24 04:05 BankkRoll

@BankkRoll is attempting to deploy a commit to the product-studio Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar May 27 '24 04:05 vercel[bot]

@BankkRoll does this fix the issue? I think the error is caused by this line: "packageManager": "[email protected]". Until you use this specific version of pnpm, it will cause this error. Can you confirm if it works?

Thanks, Arghya

itsarghyadas avatar May 27 '24 06:05 itsarghyadas

Hey @itsarghyadas can confirm it worked on my end.. - Windows

This is only the suggested version

"packageManager": "[email protected]"

This is the required minimum versions

  "engines": {
    "node": ">=20.6.1",
    "pnpm": ">=8"
  },

So this is suggesting they use 9.04 but we will accept 8 < up

BankkRoll avatar May 27 '24 13:05 BankkRoll

this is amaszing! Thanks so much for setting this up

any idea why the pnpm lockfile version is now 6.0?

image

I also noticed a lot of the package versions across the board were bumped down to older versions. Any idea why?

dillionverma avatar May 27 '24 20:05 dillionverma

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

Name Status Preview Comments Updated (UTC)
magicui ❌ Failed (Inspect) May 27, 2024 8:04pm

vercel[bot] avatar May 27 '24 20:05 vercel[bot]

Looks like rolling back the package versions caused build to fail :( image

dillionverma avatar May 27 '24 20:05 dillionverma

this is amaszing! Thanks so much for setting this up

any idea why the pnpm lockfile version is now 6.0?

image

I also noticed a lot of the package versions across the board were bumped down to older versions. Any idea why?

Hey @dillionverma i'll look some more into this tonight or tomorrow, sorry for the delay. Was out of town for holiday. I only changed the required version of pnpm, didnt downgrade any other package versions just added the new lock file as wouldn't build correctly without.. ill do some more research here tonight and try to get it fully compatible.


Looks like rolling back the package versions caused build to fail :( image

Will also look into this as well while doing so to make sure again its fully compatible. Will ping you back here to test as once i check it out 🤛

BankkRoll avatar May 28 '24 19:05 BankkRoll

The type error that UnistTree is not compatible with Node<Data> due to the missing type property should be fixed, UnistTree tree needs to extend UnistNode to fix type error.


Looks like rolling back the package versions caused build to fail :( image

Cannot build myself as there is no prisma schema to generate and test build. If you can try and let me know! Will hop in discord maybe we can connect there easier!

BankkRoll avatar May 28 '24 22:05 BankkRoll

tysm for this!!

i was trying to merge it in but the conflicts were messing me up a bit

i just ended up adding the changes directly to main!!

Tysm for surfacing this to the community and myself 🙏

https://github.com/magicuidesign/magicui/commit/37bc4adb244f18f0042ef3f3d27db23ef21900be

dillionverma avatar Jun 03 '24 05:06 dillionverma

closing for now

dillionverma avatar Jun 03 '24 05:06 dillionverma