-
Notifications
You must be signed in to change notification settings - Fork 140
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
Updating the addon #330
Comments
@fenichelar apologies for the ping. This addon seems to be incompatible with the lastest version of Ember 5 and We are relying on this addon at my work and it is blocking us from upgrading to Ember 5. I wouldn't mind putting in some work to get it compatible, but I would like to know if contributions will be accepted before putting in the effort. 😅 |
I already began to work on this when i made this issue. It works but i need to repair the CI and maybe some tests (I don't remember it was 3 months ago). It can be installed using |
Oh nice! With a bit of luck (and fixes for the tests) this could be merged upstream! |
Tracked by #334. |
Everything is kinda OK. I'm waiting for a feedback of @fenichelar before spending more time on this. |
A major overhaul is in the works including:
|
Does this make this PR #334 obsolete ? |
Hello,
Do you need assistance in transitioning this addon to use modern Ember syntax? This involves several key modifications, including:
- Eliminating deprecated Mixins.
- Discontinuing support for older Node and ember-simple-auth versions.
- Modernizing the existing Ember syntax and refreshing the dummy app.
- V2 format ?
These changes will be obviously breaking changes.
The text was updated successfully, but these errors were encountered: