Closed pradel closed 2 years ago
Latest commit: 66cfae19d2435ef15e2bf43c2350771c27787505
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
Please let me know if I can help in any way to move this forward.
Also interested to help if I can!
@Akarshit @loan-laux I am in the middle of a moving so won't have time to finish this pr, I just pushed the last commit I did some days ago but if one of you want to finish the migration feel free 🙏
Good luck with the move, @pradel!
I think I can help with this: I recently upgraded my projects to v1 and I also want to have a look at https://github.com/accounts-js/accounts/pull/1014 to see if I can make changes less drastic with v1. If we can manage to fix those fundamental issues I will also update https://github.com/accounts-js/accounts/pull/1003 to v5 and I will finally be able to switch to accounts-js. At that point I will finally be able to put more time into this and implement the missing graphql resolvers (some strategies used to work only with the rest transport).
@darkbasic this sounds fantastic, feel free to ping me on slack if you need anything!
Looks like @tedraykov has made his own PR for that: https://github.com/accounts-js/accounts/pull/1147. Super excited to have this merged!
Closing in favor of https://github.com/accounts-js/accounts/pull/1189
Closes #1079