New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix : Error logs in code segments #1968
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One quick change and the rest looks good to me!
@@ -134,13 +134,13 @@ public boolean canSetUpdateIntervals() { | |||
@Override | |||
public void setUpdateIntervals(long updateInterval, long fastestUpdateInterval) { | |||
// Do nothing. | |||
Timber.e("Can't set updateInterval on AndroidLocationClient. You should check canSetUpdateIntervals before calling this method."); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Calling setUpdateIntervals
would actually be an error in Collect code so this should be logged at level e
.
did the changes @lognaturel . Please review |
Closes #1956
What has been done to verify that this works as intended?
Why is this the best possible solution? Were any other approaches considered?
Uses
Timber.d
andTimber.i
in place ofTimber.e
.Are there any risks to merging this code? If so, what are they?
No
Do we need any specific form for testing your changes? If so, please attach one.
No
@lognaturel @grzesiek2010 please review