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

for fixing the depencie conflit with react 18 #668

Closed

Conversation

ramsesndame237
Copy link

No description provided.

@manuelLandreau
Copy link

@oliviertassinari it would be great to merge this one, thanx !

@skrln
Copy link

skrln commented Jul 12, 2022

@oliviertassinari Hey Oliver. Merging this one would be super helpful. Thanks!

@steps371
Copy link

@ramsesndame237 indenting changes in the package.json seems unnecessary, maybe that's holding up the pr?

@ramsesndame237
Copy link
Author

ramsesndame237 commented Jul 12, 2022 via email

@anton-mph
Copy link

What is ETA for React 17/18 compatible version?

@kin-nex
Copy link

kin-nex commented Jul 26, 2022

@caleb-harrelson @vanhoutenbos I believe you guys also have write permission to this repo?

Is this a PR you can review + merge?

@sanderkooger
Copy link

I would love to see this one merged!

@ndueber
Copy link

ndueber commented Aug 11, 2022

Getting this merged would be super helpful! I dont want to downgrade react to use this library!

@sanderkooger
Copy link

When will this be merged?

@sanderkooger
Copy link

sanderkooger commented Aug 16, 2022

@oliviertassinari could you merge this please?
Et tu est un Parisien no.... hahahh I would love to drink a coffee with you!

@meowtec
Copy link
Contributor

meowtec commented Aug 17, 2022

cc @meowtec

I have no permission to do that. I am not a collaborator of this project.

@zoraxl
Copy link

zoraxl commented Aug 19, 2022

Please merge this PR, really appreciate it!!

@ghost
Copy link

ghost commented Sep 8, 2022

I would love to see this merged sometime 🤞🏻

@jacobjove
Copy link

@oliviertassinari, I don't see any response here from a project maintainer. Is this package unmaintained?

@oliviertassinari
Copy link
Owner

oliviertassinari commented Sep 9, 2022

@iacobfred #558

@steps371
Copy link

@oliviertassinari, can we assume some of this repo's functionality will be included in the upcoming MUI X Carousel on the roadmap? I know it's still in the 'planning to build' phase.

I can also not find any suggestions of using this library any in MUI docs which probably means we should start looking for alternatives if no maintainer can be found

@oliviertassinari
Copy link
Owner

oliviertassinari commented Sep 16, 2022

@steps371 this project has no affiliations with MUI. At least, so far we never discussed the option to move it there. I have created this project for a mobile application I was working on as a side project. I later used it when building the mobile app of https://doctolib.com/. This was a lonnng time ago.

I agree, I think that it would be awesome to find an alternative to my repository. I would like to archive it, but it would prevent the community to collaborate on GitHub so I'm not doing it.

@steps371
Copy link

Thanks for the explanation @oliviertassinari. The main reason I started using the package was recommended by MUI documentation, so it used to have some affiliation, but got removed at some point this year.

I will do some digging on alternatives and post some suggestions for anyone interested

@ndueber
Copy link

ndueber commented Sep 22, 2022

@steps371 @oliviertassinari

Hey all, I also came to this package due to the MUI reference. I was able to use this library just by forcing npm to have some conflicting dependencies. it worked fine with react 18.

However I ended up moving away from this library due to some limitations of the package and the lack of support. I ended up using Embla Carousel. The library didn't have as many examples as this libraary, however it was customizable enough and had enough examples for me to get what i needed.

@Fermain
Copy link

Fermain commented Sep 22, 2022

I also found my way here via MUI docs and ended up using Strapi

@oliviertassinari
Copy link
Owner

I'm closing per #676, thanks for having a look at it.

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.

None yet