apps-android-commons icon indicating copy to clipboard operation
apps-android-commons copied to clipboard

Fixes Issue #5619: Add label "Congratulations, all pictures in this album have been either uploaded or marked as not for upload

Open Thejas775 opened this issue 1 year ago • 6 comments

Description (required)

Fixes #5619

What changes did you make and why? Added label "Congratulations, all pictures in this album have been either uploaded or marked as not for upload" whenever user switches off the selectorSwitch.

Thejas775 avatar Apr 18 '24 13:04 Thejas775

Here is a useful link to find unassigned bugs:

https://github.com/commons-app/apps-android-commons/issues?q=is%3Aopen+is%3Aissue+no%3Aassignee+-label%3Adebated+label%3Abug+-label%3A%22low+priority%22

:-)

nicolas-raoul avatar Apr 18 '24 13:04 nicolas-raoul

Sir is this issue resolved by this PR ?

Thejas775 avatar Apr 20 '24 11:04 Thejas775

Any idea why the text is not displayed here? Thanks! 🙂

1000044741.mp4

Will check on it sir !

Thejas775 avatar Apr 29 '24 18:04 Thejas775

@nicolas-raoul Screenshot from 2024-05-02 19-53-42 SIr earlier I thought the images.notempty will do the work. But now it seems the images are of total device and the filetered images is of the the particular folder. I tried using the notForUploadStatusDao.getNotForUploadFromImageSHA1(image.id.toString()) == null But it didnt work. If you could point me to where from can i get the status of images which are marked as not for upload or maked as upload it would be helpful.

Thanks.

Thejas775 avatar May 02 '24 14:05 Thejas775

Sorry for the delay! I don't think that using the DAO is needed. I believe that checking whether the message should be displayed or not can be done on the UI side.

nicolas-raoul avatar Aug 26 '24 06:08 nicolas-raoul

Sorry for the delay! I don't think that using the DAO is needed. I believe that checking whether the message should be displayed or not can be done on the UI side.

okay sir will try

Thejas775 avatar Sep 05 '24 12:09 Thejas775

I believe this was finished in https://github.com/commons-app/apps-android-commons/pull/6095 🙂

nicolas-raoul avatar Jan 24 '25 11:01 nicolas-raoul