-
Notifications
You must be signed in to change notification settings - Fork 141
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
Refactor sync/caching #1251
Labels
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This was one of the main reason for porting the source code to TypeScript. So now it's time to actually refactor and clean up the code that is virtually impossible to quickly debug due to extended Promise chains, some of which span 20+ functions. (Just FYI, for new contributors: when that code was written, Promise wasn't even part of JavaScript itself, but only a half-standardized spec which we used via a utility library).
My proposal would be to use async/await as much as possible instead.
I have created a project, and added all sync-related issues that I could glance in the list. Maybe we can address some or most of those on the way. But it's mostly to get an overview of open issues in that area, and what we can fix or improve either specifically on the way, or by cleaning up code and making it more robust.
https://github.com/remotestorage/remotestorage.js/projects/1
The text was updated successfully, but these errors were encountered: