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

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

Open
nicolas-raoul opened this issue Mar 10, 2024 · 10 comments · May be fixed by #5696

Comments

@nicolas-raoul
Copy link
Member

In the custom picker, with "Show already actioned pictures" disabled, this is how the screen looks after all pictures have been either uploaded or marked as not for upload:

Screenshot_20240310-162611.png

Ideally, it should instead display a message such as:

"Congratulations, all pictures in this album have been either uploaded or marked as not for upload."

@neeldoshii
Copy link
Contributor

Can we either display "An vector" like this?

If no image is left to upload or masked as not for upload, we can display svg image & display text like

  1. Custom Picker finalised: Every image is now either uploaded or marked.
  2. Album perfection achieved: Every photo uploaded or sorted.
    I don't think congratulations is right word as if some person has marked all images not for upload will get text congratulation might be incorrect from UX copywriting pov.

image
Credits

What do you think about it @nicolas-raoul. I can tackle this while my wikimedia id is banned.

@nicolas-raoul
Copy link
Member Author

@neeldoshii Sorry for the delay! Yes vetor image would be nice, if possible.

"Congratulations": Even marking all pictures as not for upload is good, so I think it is a good opportunity to make people feel good. :-) Maybe something like "Good job! [...]"

Maybe it could even suggest the user to perform peer reviews or go take nearby pictures, with buttons to launch these activities.

@starrain1031
Copy link

I would like to try to solve this problem. Would you like to assign it to me?

@Thejas775
Copy link

I would like to try to solve this problem. Would you like to assign it to me?

Are you still working on this issue ?

@starrain1031
Copy link

I would like to try to solve this problem. Would you like to assign it to me?

Are you still working on this issue ?

Yes. However, based on my limited skills, it might take me sometime.

@nicolas-raoul
Copy link
Member Author

@Thejas775 Thanks for your enthusiasm, however better work on unassigned bugs (the app has enough bugs for everybody haha) next time.

@starrain1031 Sorry for unusual workflow this time. Maybe review Thejas775's pull request, improve it if possible, then work on a second bug if you want? :-)

@Thejas775
Copy link

I'm really sorry but this was my first contribution to open source as well. This was a good first issue that I thought I could solve.

@nicolas-raoul
Copy link
Member Author

@Thejas775 That's understandable :-) You might want to read https://github.com/commons-app/commons-app-documentation/blob/master/android/Volunteers-welcome!.md#developers which has the link to unassigned bugs. I hope you will enjoy fixing other bugs :-)

@Thejas775
Copy link

@Thejas775 Thanks for your enthusiasm, however better work on unassigned bugs (the app has enough bugs for everybody haha) next time.

@starrain1031 Sorry for unusual workflow this time. Maybe review Thejas775's pull request, improve it if possible, then work on a second bug if you want? :-)

Will follow the workflow from next time.

@Thejas775
Copy link

@Thejas775 That's understandable :-) You might want to read https://github.com/commons-app/commons-app-documentation/blob/master/android/Volunteers-welcome!.md#developers which has the link to unassigned bugs. I hope you will enjoy fixing other bugs :-)

Yeah sure seems intresting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment