Skip to content
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

Fix-Peer Review Buttons not get disabled after voting an Image for deletion #5519

Merged

Conversation

shashankiitbhu
Copy link
Contributor

Description (required)

Fixes #5516

What changes did you make and why?
Peer Review Buttons ("Yes" and "No") gets disabled to after nominating an Image for deletion to avoid Incorrect Input from user.

Tests performed (required)

Tested 4.2.1-debug-main on Xiaomi 11 Lite NE with API level 33

Screenshots (for UI changes only)

WhatsApp.Video.2024-02-05.at.6.29.07.PM.mp4

Copy link
Member

@nicolas-raoul nicolas-raoul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed a bug:

  1. At question Does this follow the rules of copyright?, tap No. The buttons are correctly disabled.
  2. Continue reviewing the next picture.
  3. At question Does this follow the rules of copyright?, the buttons are incorrectly already disabled, before I have a chance to tap any for that picture.

Screenshot_20240214-225318.png

@shashankiitbhu
Copy link
Contributor Author

@nicolas-raoul thanks for reviewing and finding this bug, will make changes to fix this

@shashankiitbhu
Copy link
Contributor Author

@nicolas-raoul If these changes I made works then can you review this PR please, thanka

@shashankiitbhu
Copy link
Contributor Author

@nicolas-raoul Just Pinging In case you missed it, Please Review

@nicolas-raoul
Copy link
Member

Apologies, I have been trying to peer review until I see a bad image to delete, but the uploads are all good...

Copy link
Member

@nicolas-raoul nicolas-raoul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to be working.

After nominating for deletion I repeatedly tapped on Yes and No but the buttons did nothing, they appear to be greyed out. Also, I could review more pictures afterwards normally.

screen-20240327-214247.mp4

@nicolas-raoul nicolas-raoul merged commit 4c43bf2 into commons-app:main Mar 27, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Peer Review Buttons are not disabled while next image is Loading After Nominating it for deletetion
2 participants