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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix crash in splash when offline #367
Conversation
1锔忊儯 1锔忊儯 |
@@ -73,7 +71,8 @@ private void loadSessionsForCache() { | |||
if (isFinishing()) return; | |||
startActivity(MainActivity.createIntent(SplashActivity.this)); | |||
}) | |||
.subscribe(); | |||
.subscribe(objectObservable -> Timber.tag(TAG).d("Succeeded in loading sessions."), |
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.
Can you change valuable to observable
from objectObservable
?
In fact, lambda variable should be more short name like o
, t
.
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.
Succeeded to load sessions.
is better?
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.
I'll fix it soon! Thanks 馃檱
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.
I think that "succeed to" means "寮曘亶缍欍亹"... 馃
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.
Oh I didn't know that 馃檱 Thanks
LGTM! 馃挴 |
Issue
none
Overview (Required)
Since the app has no error handling in
loadSessionsForCache()
, it crashes when fails to load sessions in splash. 馃挜I added only logs using Timber.
I think that we need to implement session reloading function in the future.
Links
none
Screenshot
none