Added ability to enable metrics manager after it was disabled #451
Conversation
|
@bmourat, |
| if (disabled) { | ||
| [self unregisterObservers]; | ||
| } else { | ||
| [self registerObservers]; | ||
| [self startManager]; |
There was a problem hiding this comment.
No need to generate a new session UUID
There was a problem hiding this comment.
We might even not have a session id, if startManager was not called, and since we a starting new session each time app is in background mode for 20 seconds or more, i don't think we need to worry about starting new session here. Besides it is rather logical to start new session each time after enabling metrics manager.
There was a problem hiding this comment.
I agree that it makes sense to start a new session in case the feature was disabled and then enabled again.
No description provided.