Error installing node modules

tianchengc / exable-admin

Other

1 stars 1 forks source link

Error installing node modules #31

Closed swapnils118 closed 1 day ago

swapnils118 commented 3 days ago

Weird version type errors while installing node modules once we take clone from main branch.

dgtlmonk commented 3 days ago

I believe this is caused by node-sass.

I've encountered this a while ago, tried to fix but no success (unless I remove node-sass from dependencies - but ran into other issues.

I've issued a similar concern, I believe @tianchengc is working on this.

swapnils118 commented 3 days ago

Thanks for your response @dgtlmonk and yes we both are working towards fixing this. Can you tell me from your experience on how did you make the code work then in order to start developing/contributing further? Your any response is much appreciated. Thanks

dgtlmonk commented 3 days ago

@swapnils118 I am still stuck in not being able to run this codebase.

@tianchengc, when you are free; could you clone this repo and try to run yourself if you don't get this same issue when installing fresh dependencies?

swapnils118 commented 3 days ago

If we either remove node-sass or run npm install --peer-legacy-peer-deps, we are getting errors as:

dgtlmonk commented 3 days ago

Yeah, looks like this "glitch" is cross-dependency issues and is tedious to fix due to dependencies version compatibilities. I suggest, we start from scratch while there's not much code in here.

I also suggest, we add the dependencies as needed - i noticed we have too many dependencies that are unnecessary upfront.

I can help set a fresh repo if needed.

swapnils118 commented 3 days ago

Yes I also support this side. But still I will dig deep into these issues and try to find the catch.

tianchengc commented 2 days ago

Yeah, looks like this "glitch" is cross-dependency issues and is tedious to fix due to dependencies version compatibilities. I suggest, we start from scratch while there's not much code in here.

I also suggest, we add the dependencies as needed - i noticed we have too many dependencies that are unnecessary upfront.

I can help set a fresh repo if needed.

Totally, @dgtlmonk please go for it! Could you raise a pr for this? That would solve a big headache for us!

tianchengc commented 1 day ago

@swapnils118 could you validate again with latest main branch, @dgtlmonk has fixed it

swapnils118 commented 1 day ago

@tianchengc Yes I am back to work. Let me have a look.

swapnils118 commented 1 day ago

@Take care The problem is solved. A big thanks to @dgtlmonk

tianchengc commented 1 day ago

Thanks for confirming!