search

openambitproject / openambit

openambit
281 stars 82 forks source link

Failed to automatically sync log entries to movescount. #106

Closed superdupersvard closed 8 years ago

superdupersvard commented 8 years ago

I was not able to get openambit to automatically sync log entries to movescount.com when the watch was connected to openambit.

I found out that in MainWindow::deviceDetected() that settings.beginGroup() was called and then SyncStart() was called, that also called settings.beginGroup() before settings.endGroup() has been called for the first settings.beginGroup(). This resulted in that the settings were not correctly read and that no sync with movescount was done.

I have made a fix for this in my branch ( https://github.com/superdupersvard/openambit ). Github is new to me, so I am unsure how to proceed to get this fix in to the master branch.

benedetto commented 8 years ago

The help pages are your friend ;-)

Consider the help pages on proposing changes in general and especially the page on creating pull requests.

paddy-hack commented 8 years ago

Seeing that you opened #107 and that I just merged that, I guess this question can be closed