-
Notifications
You must be signed in to change notification settings - Fork 0
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.
Thanks!! Nice fixes! I have left two comments, but otherwise all good!
|
||
Log.e("INFO: Should Access", ((InternetState.hasAccessedInternetOnce(this))).toString()) | ||
PersistanceSetter.setPersistance() | ||
Database.get() |
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.
why do you need Database.get() here if the result is not saved anywhere?
Co-authored-by: kamilababayeva <[email protected]>
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.
LGTM
Code Climate has analyzed commit f9af34a and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (80% is the threshold). This pull request will bring the total coverage in the repository to 85.1% (-0.8% change). View more on Code Climate. |
Fixed Bugs