Skip to content

use subfolders for attachments#6127

Open
sowjanyakch wants to merge 2 commits intomasterfrom
feat/6034/conv-subfolders
Open

use subfolders for attachments#6127
sowjanyakch wants to merge 2 commits intomasterfrom
feat/6034/conv-subfolders

Conversation

@sowjanyakch
Copy link
Copy Markdown
Contributor

@sowjanyakch sowjanyakch commented Apr 27, 2026

Resolve #6034

For NC34, we use subfolders for attachments per conversation.

  1. We need to get the draft folder from the endpoint, predicted filenames in case of conflict from the endpoint ....../attachment/folder endpoint
  2. Upload attachments to the draft folder using the predicted filenames.
  3. Call ....../attachment endpoint to finalise the upload and to post a chat message.

🚧 TODO

  • ...

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔖 Capability is checked or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

@github-actions
Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/24994930982/artifacts/6660687770
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@sowjanyakch sowjanyakch force-pushed the feat/6034/conv-subfolders branch from 981ffb3 to ae2eb26 Compare April 27, 2026 17:02
@sowjanyakch sowjanyakch self-assigned this Apr 27, 2026
@sowjanyakch sowjanyakch added 3. to review Waiting for reviews labels Apr 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/25008606201/artifacts/6666835121
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

Comment thread app/src/main/java/com/nextcloud/talk/utils/CapabilitiesUtil.kt Outdated
@sowjanyakch sowjanyakch force-pushed the feat/6034/conv-subfolders branch from ae2eb26 to 4867f77 Compare May 5, 2026 17:28
@sowjanyakch sowjanyakch marked this pull request as draft May 5, 2026 17:29
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

APK file: https://github.com/nextcloud/talk-android/actions/runs/25391802249/artifacts/6813658464
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@sowjanyakch sowjanyakch force-pushed the feat/6034/conv-subfolders branch from 4867f77 to c65d41d Compare May 6, 2026 16:52
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

APK file: https://github.com/nextcloud/talk-android/actions/runs/25449003837/artifacts/6836474927
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
@sowjanyakch sowjanyakch force-pushed the feat/6034/conv-subfolders branch from c65d41d to 55aff01 Compare May 6, 2026 17:11
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

APK file: https://github.com/nextcloud/talk-android/actions/runs/25449918586/artifacts/6836945189
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
@sowjanyakch sowjanyakch marked this pull request as ready for review May 6, 2026 17:34
@sowjanyakch sowjanyakch requested a review from mahibi May 6, 2026 17:34
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

APK file: https://github.com/nextcloud/talk-android/actions/runs/25450704773/artifacts/6837404172
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

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

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use subfolder for Talk attachments, one per conversation

2 participants