-
Notifications
You must be signed in to change notification settings - Fork 27
Add warning message when post is rejected #23
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
Add warning message when post is rejected #23
Conversation
Codecov Report
@@ Coverage Diff @@
## master #23 +/- ##
==========================================
- Coverage 39.02% 34.78% -4.25%
==========================================
Files 3 3
Lines 41 46 +5
==========================================
Hits 16 16
- Misses 22 27 +5
Partials 3 3
Continue to review full report at Codecov.
|
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.
Thanks for jumping on this 👍
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.
🎉
@nathanaelhoun Heads up that there is a merge conflict |
@hanzei Thanks for the heads up! This has been fixed :-) |
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.
Tested and passed
- Rejected post UI is clear
- Tested on desktop and mobile
- No issues found
LGTM!
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.
LGTM
It looks like in the post-rejected.gif that the post is still posted, above the ephemeral message. Is this post supposed to be removed from the user's UI after being rejected? |
@mickmister You are right, I shouldn't have cut the right side of the capture. Should I submit a PR to replace with this one? The post isn't posted, and it diseappear when reloading / clicking "cancel" |
@nathanaelhoun It would seem that webapp itself displays the Up to you whether you want to create a PR for the gif change :) |
@mickmister Exactly, the webapp itself provide the It was already the case before, but the user wasn't able to distinguish this from a network issue: now he knows he has to watch his language ;-) I know best is the enemy of good, but I want to make the work correctly: I have submitted #29 :D |
Summary
config.json
,README.md
updated with gifs.Ticket Link
Fixes #21