mc96107 / mcnotes

Organized, note taking app with wiki connections and markdown syntax. Syncs with Dropbox and remoteStorage. https://mcnotes.5apps.com/
6 stars 0 forks source link

Update remoteStorage.js to 0.10.0-beta #1

Closed raucao closed 9 years ago

raucao commented 10 years ago

I saw you already tried to update rs.js, but reverted afterwards. What's the problem exactly? I'd gladly help.

mc96107 commented 10 years ago

Thanks for offering to help!
In doc.js I update a list when event sync-done. After I updated remote storage it was not working using dropbox. I would appreciate any help on this!

raucao commented 10 years ago

Dropbox support is not official yet. Unfortunately a half-working solution has been merged before we introduced the new review/qa process.

I would highly recommend not to use that experimental feature in production yet, but hide it to end users and report any Dropbox bugs you encounter to help us develop the feature. However, we haven't even released Dropbox support as a beta-quality feature yet, so you might want to wait even with that until it's officially a thing (likely in a few weeks from now, as @michielbdejong has it on his milestone list for NLnet). Or maybe you even want to help out with the code?

michielbdejong commented 10 years ago

yes, for dropbox support you need at least to replace the src/dropbox.js file from the master branch with the one from the backend-tests branch: https://github.com/remotestorage/remotestorage.js/blob/backend-tests/src/dropbox.js and then follow the build instructions to produce a new remotestorage.js

mc96107 commented 10 years ago

ok thanks for the advice, I have built it and am running some tests locally.

mc96107 commented 10 years ago

Just pushed here and on https://mcnotes.5apps.com/ a (hopefully) working version. Please inform on any issues found! It supports remotestorage.js 0.10.0-beta and Dropbox sync, based on https://github.com/mc96107/mcnotes/issues/1#issuecomment-39640953

raucao commented 10 years ago

Cool!

I run into remoteStorage.sync is undefined errors.

Looking at our changelog, I think someone forgot to add the information that there is no public sync function anymore. Sorry about that!

Creating, updating, and deleting objects immediately pushes the changes, so you don't need to sync after using it. And all getX functions now support a maxAge parameter, as well as sync any changes when the local cache is outdated.

raucao commented 10 years ago

Hmm, weird. Reconnected and used it some more and I can't reproduce it. Everything seems to be working just fine now. Maybe I had some old code cached or sth.

mc96107 commented 10 years ago

oops just closed it by mistake..

I am using remoteStorage.sync.done as a flag to know when sync is done so that I can refresh the list of notes. Used to do it with sync-done event but this is not in the new version. Glad that everything works, and hope it keeps working :)

raucao commented 10 years ago

FYI: we just released 0.10.0-beta2, which fixes an important bug we found during testing: http://community.remotestorage.io/t/remotestorage-js-0-10-0-beta2-released/185

mc96107 commented 10 years ago

ok updated to remotestorage 0.10.0-beta2. Also added new feature (Press right/left keyboard arrow or swipe right-left to move between notes)

raucao commented 10 years ago

Cool!

michielbdejong commented 10 years ago

off the top of my head, it's (now) remoteStorage.sync.sync(); you should not need to call it explicitly, but it may be useful for debugging. see also remoteStorage.sync._running and remoteStorage.sync._tasks to get more info on what sync is working on.