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
WEB-566: [Android] UI for flagged comment #1643
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1643 +/- ##
=========================================
Coverage 78.43% 78.43%
Complexity 1862 1862
=========================================
Files 353 353
Lines 16573 16573
Branches 2039 2039
=========================================
Hits 12999 12999
Misses 2406 2406
Partials 1168 1168
Continue to review full report at Codecov.
|
android:id="@+id/flagged_message" | ||
style="@style/CommentCardFlaggedMessage" | ||
android:visibility="gone" | ||
android:text="@string/this_comment_is_under_review" |
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.
Question here @leighdouglas, is there any ticket prepared for translations?
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.
Good point! I'll create that 👍🏽
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.
Minor comments, lgtm! 👍
d289944
to
eeb413b
Compare
…tin/PAY-1693-integrate-paymentsheet * 'master' of github.com:kickstarter/android-oss: WEB-605: Update string for flagged comments (#1650) NTV-563 : Migrate Message related viewmodels classes to Kotlin (#1649) WEB-567: Show flagged comment UI/state when pending T&S review (#1648) - Release 3.3.0 Comments from canceled backers with no replies do not appear in the list of comments (#1645) NTV-561: Update dependencies (#1646) WEB-566: [Android] UI for flagged comment (#1643) NTV-560: Updated gems (#1644) PAY-1716: Update stripe SDK (#1642) NTV-556 : Migrate Creator Dashboard viewmodels classes to Kotlin (#1639) WEB-532 : [Android] Canceled backer: Remove "show comment" link from comment tombstones (#1641)
📲 What
Created comment card UI for flagged comment state.
🤔 Why
As a part of comment moderation updates, we want to hide potentially abusive comments from users while they are under review.
🛠 How
Created the new UI state for flagged comments, updated the comment card layout and class, and created screenshot tests.
👀 See
I censored the usernames as these were tests and those comments were not actually flagged.
📋 QA
Story 📖
WEB-566: [Android] UI for flagged comment