Combine `startTracking` and `startSelfTracking` methods
ravikanththummala opened this issue · 1 comments
ravikanththummala commented
Discussed in #8
Originally posted by ravikanth-tummala July 22, 2021
Currently
- The
startSelfTracking()
does not need user sessions which are basically created usingcreateUser()
orgetUser()
methods. - And, they track location locally and does not publish location to Roam.ai servers. Even if the
publishSave()
methods are being called, they will throw an error. - In order to publish location updates to server, along with
createUser()
orgetUser()
methods, we have usestartTracking()
instead ofstartSelfTracking()
.
Proposed
- We can combine the
startTracking()
andstartSelfTracking()
into a single method and name it asstartTracking()
- If the user session is not available ie.
createUser()
orgetUser()
not being called, the location update works locally and behaves similar tostartSelfTracking()
- And if the user session are enabled using
createUser()
orgetUser()
methods andstartTracking()
will still continue to update locations locally unless until thepublishSave()
orpublishOnly()
methods are called - Along with this the
stopTracking()
andstopSelfTracking()
into a single method and name it asstopTracking()
jothipriyadharshanr commented
Resolved