-
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-2003:UX β Show pending comment dialog #1329
Conversation
# Conflicts: # app/src/main/res/values/strings.xml
@@ -18,3 +19,41 @@ fun Context.registerActivityLifecycleCallbacks(callbacks: Application.ActivityLi | |||
this.registerActivityLifecycleCallbacks(callbacks) | |||
} | |||
} | |||
|
|||
fun Context.showAlertDialog( |
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.
β€οΈ
β¦g' into hadia/Show_pending_comment_dialog
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.
π great work @hadia
Codecov Report
@@ Coverage Diff @@
## master #1329 +/- ##
=========================================
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.
|
β¦tin/NT-2125-first-test * 'master' of github.com:kickstarter/android-oss: - Removed sharedUserId property NT-2003:UX β Show pending comment dialog (#1329) NT-1996 :UI/UX β Error loading thread (#1328) NT-2093: Skip first emitted value from paginated data to avoid initially showing empty state showing on root comments screen (#1323) NT-2124: Shot 5.11.0 library integration (#1324) # Conflicts: # app/build.gradle # app/src/androidTest/java/com/kickstarter/screenshoot/testing/di/AndroidTestApplicationModule.kt # app/src/androidTest/java/com/kickstarter/screenshoot/testing/ui/components/CommentCardShotTest.kt # app/src/internalDebug/AndroidManifest.xml
π² What
Show pending comment dialog
π€ Why
Add confirmation dialog when users have a failed post or something written in the comment composer and try to
π How
Create alert Dialog
![image](https://user-images.githubusercontent.com/1075310/126195103-3cac50fe-e00a-4fcb-b48a-0d85c90831ce.png)
![image](https://user-images.githubusercontent.com/1075310/126195197-24cf270d-1293-43a6-a32a-cc80b8b3c0c7.png)
Check if have pending comments in case of back and pull to refresh
π See
device-2021-07-19-202828.mp4
π QA
Story π
https://kickstarter.atlassian.net/browse/NT-2003