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.

Sign up for GitHub

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

Jump to bottom

Migrate to Ionic 7 due to Bug with IonRange #25

Open
JesJehle opened this issue Mar 14, 2023 · 3 comments · May be fixed by #63
Open

Migrate to Ionic 7 due to Bug with IonRange #25

JesJehle opened this issue Mar 14, 2023 · 3 comments · May be fixed by #63
Assignees
@mmaelicke

Comments

@JesJehle
Copy link
Collaborator

JesJehle commented Mar 14, 2023 •
edited
Loading

In the Old version of Ionic, there is a known Bug with IonRange, which I could not solve otherwise:
ionic-team/ionic-framework#21101
The bug is solved in Ionic 7.
I tried it, and it works. Although Ionic 7 is still in Beta I would suggest upgrading.

Ionic 7 supports React 17+. Update to the latest version of React:
npm install react@latest react-dom@latest
Update to the latest version of Ionic 7:
npm install @ionic/react@next @ionic/react-router@next

What do you think?
@mmaelicke
Copy link
Member

I would suggest to do that in an extra branch and test everything as Ionic updates usually break something...

@JesJehle
Copy link
Collaborator Author

I already installed it in my branch. It's working fine, but I will make further tests.

@mmaelicke
Copy link
Member

Do this after merge of #48

@mmaelicke mmaelicke linked a pull request Apr 9, 2023 that will close this issue
Open
@JesJehle JesJehle modified the milestone: 1. Open Testing Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mmaelicke mmaelicke

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

update ionic to 7 hydrocode-de/mathislewald
2 participants
@mmaelicke @JesJehle