search

nitrotasks / nitro

The beautiful way to get things done.
https://nitrotasks.com
GNU Affero General Public License v3.0
518 stars 66 forks source link

Sync #248

Closed stayradiated closed 10 years ago

stayradiated commented 10 years ago

The sync code needs a big clean up.

The queue optimisation code can be refactored into three separate functions handling create, update and destroy events.

The localStorage code has already been separated from the sync code.

It also needs to be refactored to work with the Base framework.

stayradiated commented 10 years ago

Sync Bugs:

[x] Updating a task doesn't work. Server can't find the task.
[x] Sometimes when you create a task it is emitted to the server but not actually added to a list.
[x] Client does not sync queue when app is started.

[x] Tasks sometimes do not show up in lists
[x] The first time that queue.sync() is run, the refresh() does not update the UI
[x] Marking a task as completed doesn't work.
stayradiated commented 10 years ago

I've just a fixed a small bug that was causing lots of problems with Sync - the model id was being deleted. So yeah, hopefully most of the bugs will also be fixed. Going to try out soon.

After that I should be able to write some proper tests for each individual model, and see if they are working with sync properly.