Beeminder integration bugfix for when TT is being run on multiple machines #35

Merged
merged 3 commits into from Jan 16, 2014

Conversation

Projects
None yet
2 participants
@pmyteh
Contributor

pmyteh commented Jan 15, 2014

This is a bugfix, coupled with a FIXME on ttlogmerge which was the subject of my last merge request. It is part of an ongoing effort to make TagTime co-operate seamlessly when run on multiple machines at once.

Tom Nicholls and others added some commits Dec 30, 2013

Fix bug in handling of multiple datapoints for a given day on the ser…
…ver.

For TT there should be only one datapoint per day on the beeminder server.
The way things were being handled, duplicates were being deleted from the
data fetched from the server, but the resulting empty entry in the array
was sometimes causing a (different) duplicate to be re-created later in the
script. This bug would only normally show up under circumstances where two
copies of TagTime were writing to the same graph, for example when it is being
run on two machines simultaneously.

This commit changes the deletion method to result in no duplicate entries
being created.

dreeves added a commit that referenced this pull request Jan 16, 2014

Merge pull request #35 from pmyteh/beeminder
Beeminder integration bugfix for when TT is being run on multiple machines

@dreeves dreeves merged commit 6aeaa6b into tagtime:master Jan 16, 2014

@pmyteh pmyteh deleted the pmyteh:beeminder branch Jan 16, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment