-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Fix/upload limit #3101 #6500
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?
Fix/upload limit #3101 #6500
Conversation
…updated warning dialog strings, and ensured thumbnail bar consistency
app/src/main/java/fr/free/nrw/commons/customselector/ui/adapter/ImageAdapter.kt
Outdated
Show resolved
Hide resolved
|
@nicolas-raoul Changed from 5 image limit to 20 image limit as per #3101 (comment) |
app/src/main/java/fr/free/nrw/commons/customselector/ui/adapter/ImageAdapter.kt
Outdated
Show resolved
Hide resolved
|
@nicolas-raoul - Made the required changes. |
|
I am travelling (now Hyderabad) so I can't test, but I hope I will be able to do it on Wednesday. Or someone might do it before me. Thank you for your understanding! 🙂 |
Oh, Hyderabad! Wishing you a pleasant journey. I hope you get the chance to explore and enjoy the beauty of the city. |
|
✅ Generated APK variants! |
Description (required)
Fixes #3101
Implemented a strict 20-image selection and upload limit in the Wikimedia Commons app to prevent server overload and align with user feedback. Modified
CustomSelectorActivity.ktandImageAdapter.ktto restrict selection to 5 images, updatedUploadActivity.ktto process only 5 images, added a close button initem_custom_selector_image.xmlfor deselection, updatedcustom_selector_limit_dialog.xmlandstrings.xmlfor a clear 5-image limit message, and ensuredThumbnailsAdapter.ktdisplays only 5 images in the upload UI thumbnail bar.