-
Notifications
You must be signed in to change notification settings - Fork 1.3k
fixes the bug regarding copy to subsequent media when near a wikidata place. #5163
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
base: main
Are you sure you want to change the base?
Conversation
I just tested, it indeed solves the issue. |
@nicolas-raoul I will surely look into this and see if I can do something about it. |
So far I have not managed to replicate on the master branch the behavior I just described above. |
@nicolas-raoul What should be the next step from my side then? |
As the problem seems to happen only in this pull request branch, there are chances that it is triggered by the pull request's change. |
What I understand from this is that if I have 5 images and first two of them are of a wikidata place then the copy to subsequent media should only copy captions to the images which are of a wikidata place(i.e first 2 images) and not to the last 3 images that are of a non-wikidata place, but the pull request change is making the issue of copying the captions to all the 5 images whether it is of a wikidata place or not , right? |
What I meant is: First I uploaded a set of 2 images of the Eiffel Tower. The copy button worked great, and the upload finished successfully. |
@nicolas-raoul Apologies for responding so late, I was caught up in my semester exams. I have understood the issue and will look into it. |
@ayushanand308 No problem, good luck for the exams! Sure, feel free to submit a proposal, please edit it once your pull request gets merged. |
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.
Did you get a chance to look into this? :-)
Description (required)
Fixes #5057
What changes did you make and why?
Changed the value of "showNearbyFound"(in the method onNearbyPlaceFound) from true to false that may have been conflicting with the method "onBecameVisible" whenever next/previous button was clicked.
Tests performed (required)
Tested ProdDebug on Xiaomi Redmi 9 with API level 29.