Fix/external upload limit

[Kota-Jagadeesh]

This version is enhanced from the initial proposal (#6555) and includes key refactorings:

1. user-Friendly truncation (instead of Hard Block):

• In UploadActivity.kt, the upload list is now checked for size after file processing.
• If fileCount > 20, the list is immediately truncated to the MAX_EXTERNAL_UPLOAD_COUNT.

A non-blocking toast is shown to the user informing them that only the first 20 files were loaded, ensuring workflow continuity with clear feedback.

2. Constant Centralization (Refactoring):

• The hardcoded limit (20) was removed from CustomSelectorActivity.kt and centralized into a dedicated shared constant: fr.free.nrw.commons.filepicker.Constants.MAX_EXTERNAL_UPLOAD_COUNT.

Both UploadActivity.kt and CustomSelectorActivity.kt now correctly reference this centralized constant, improving code maintainability and adherence to DRY principles.

3. New String Resource:

• Added R.string.multiple_upload_limit_truncated_message to inform the user about the truncation.

[Kota-Jagadeesh]

@nicolas-raoul @RitikaPahwa4444 Any update on this PR ?

[Kota-Jagadeesh]

Hi @nicolas-raoul, just checking in to see if this PR is still under review. Please let me know whenever you have time if there’s anything I should update or change. Thanks!

[nicolas-raoul]

Sorry I am super busy this start of year, hopefully someone else can review? 😥

[Kota-Jagadeesh]

No worries at all @nicolas-raoul, totally understand 😊
I’ll wait for others to review it. I was just following up on some of my older PRs since my open PR count has been increasing. Thanks for letting me know!

[github-actions]

✅ Generated APK variants!

• 🤖 Download betaDebugAPK
• 🤖 Download prodDebugAPK

[Kota-Jagadeesh]

@nicolas-raoul Closing this PR as the issue is fixed in my updated PR #6689 along with issue #3101