Skip to content

reply to a thread should navigate to a thread#5836

Merged
mahibi merged 1 commit intomasterfrom
fix/5832/threadReply
Feb 12, 2026
Merged

reply to a thread should navigate to a thread#5836
mahibi merged 1 commit intomasterfrom
fix/5832/threadReply

Conversation

@sowjanyakch
Copy link
Contributor

@sowjanyakch sowjanyakch commented Feb 10, 2026

fix #5832

There should be no option to quote a thread. Reply to a thread should navigate to the thread (same behavior is seen in ioS and web).

Had to fix this because it impacts scheduled message. Quote to a thread should not be scheduled or sent (both cases will cause unnecessary network call to schedule endpoint or send endpoint).

🚧 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?)

@sowjanyakch sowjanyakch requested review from mahibi and rapterjet2004 and removed request for mahibi February 10, 2026 15:58
@sowjanyakch sowjanyakch self-assigned this Feb 10, 2026
@sowjanyakch sowjanyakch added the 3. to review Waiting for reviews label Feb 10, 2026
@mahibi mahibi force-pushed the fix/5832/threadReply branch from 6a084aa to 31a2e48 Compare February 12, 2026 11:18
@mahibi mahibi added this to the 23.1.0 milestone Feb 12, 2026
@mahibi
Copy link
Collaborator

mahibi commented Feb 12, 2026

/backport to stable-23.0

Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
@github-actions
Copy link
Contributor

Codacy

Lint

TypemasterPR
Warnings9898
Errors00

SpotBugs

CategoryBaseNew
Bad practice66
Correctness1010
Dodgy code5454
Internationalization33
Malicious code vulnerability33
Performance44
Security11
Total8181

@github-actions
Copy link
Contributor

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/5836.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@mahibi mahibi merged commit 719169e into master Feb 12, 2026
15 of 18 checks passed
@mahibi mahibi deleted the fix/5832/threadReply branch February 12, 2026 12:25
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.

"Reply" for thread message should navigate to the thread

2 participants