-
Notifications
You must be signed in to change notification settings - Fork 992
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
NT-1997:UI/UX β Error loading more replies #1313
Conversation
# Conflicts: # app/src/main/java/com/kickstarter/ui/adapters/RepliesAdapter.kt # app/src/main/java/com/kickstarter/viewmodels/ThreadViewModel.kt # app/src/main/res/layout/activity_thread_layout.xml
# Conflicts: # app/src/main/java/com/kickstarter/ui/activities/ThreadActivity.kt # app/src/main/java/com/kickstarter/ui/adapters/RepliesAdapter.kt # app/src/main/java/com/kickstarter/viewmodels/ThreadViewModel.kt # app/src/main/res/layout/activity_thread_layout.xml
Codecov Report
@@ Coverage Diff @@
## master #1313 +/- ##
=========================================
Coverage 74.12% 74.12%
Complexity 739 739
=========================================
Files 221 221
Lines 6666 6666
Branches 406 406
=========================================
Hits 4941 4941
Misses 1589 1589
Partials 136 136 Continue to review full report at Codecov.
|
@@ -50,6 +50,9 @@ interface ThreadViewModel { | |||
|
|||
fun isFetchingReplies(): Observable<Boolean> | |||
fun loadMoreReplies(): Observable<Void> | |||
|
|||
/** Display the bottom pagination Error Cell **/ |
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.
On threads is displayed on top, modify the comment
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.
Done
this.internalError | ||
.compose(Transformers.combineLatestPair(onCommentReplies)) | ||
.filter { | ||
// it.first.second && |
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.
Remove comment
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.
Done
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.
- Some picky comments, but is good to go on my end :) π
β¦ure/NT-2059-update-stripe-sdk * 'master' of github.com:kickstarter/android-oss: NT-1997:UI/UX β Error loading more replies (#1313) NT-2095:View Replies show and hide with scrolling (#1312) NT-2053:UX β Add pagination to replies (#1311) NT-2028: Navigation β Back button should slide view out #1310 NT-1958: UX β Post a reply (#1305) NT-2071: Update exoplayer, mockito, and timber dependencies to latest versions (#1306) Set the name as an empty string to prevent the app from crashing. (#1309) NT-2069:Refactor Comment Pagination code (#1308) NT-2005: Update Dependencies to Latest Version (#1304) NT-2069:Refactor Comment Pagination code (#1303) NT-1958: UX β Post a reply (#1301)
π² What
handle problems loading more replies
π€ Why
show pagination Error loading extra pages of replies
π How
same as comments we change the view more to the error message and its clickable to retry
π See
Screen_Recording_20210706-154610_Kickstarter.mp4
π QA
Story π
https://kickstarter.atlassian.net/browse/NT-1997