-
Notifications
You must be signed in to change notification settings - Fork 141
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
Created sessions page #66
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.
Reviewed and put some comments!
Disposable disposable = viewModel.getSessions(languageId, getContext()) | ||
.subscribe( | ||
this::renderSessions, | ||
throwable -> Timber.e(TAG, "Failed to show sessions.", throwable) |
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 arguments of Timber.e()
should be (Throwable, String, Object...)
.
You can use Timber.tag(String)
to set a log tag before calling e()
.
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 see. I'll fix this! Thanks!
} | ||
|
||
switch (topic.id) { | ||
case 1: |
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.
Magic number 😏
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.
Replaced to static fields.
Reviewed! |
@KeithYokoma Addressed! Can you review again? |
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.
Issue
#11
Overview (Required)
Structure
To be honest, I wanted to use
RecyclerView + custom LayoutManager
.But it's difficult to catch up in a short time.
So I implemented like this.
I'm using twoway-view which has
SpannableGridLayoutManager
which providessetColSpan()
andsetRowSpan()
to RecyclerView.In fact, the dummy item is filled as blank time slot like below.
I'll try to improve this page by using custom LayoutManager on another PR 💪
Links