Skip to content

Edit mentions#5072

Merged
sowjanyakch merged 17 commits intomasterfrom
edit_messages
Jul 8, 2025
Merged

Edit mentions#5072
sowjanyakch merged 17 commits intomasterfrom
edit_messages

Conversation

@sowjanyakch
Copy link
Copy Markdown
Contributor

@sowjanyakch sowjanyakch commented Jun 23, 2025

🖼️ Screenshots

🚧 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 marked this pull request as draft June 23, 2025 08:22
@sowjanyakch sowjanyakch changed the title Edit messages Edit mentions Jun 23, 2025
@sowjanyakch sowjanyakch self-assigned this Jun 23, 2025
@sowjanyakch sowjanyakch added the 3. to review Waiting for reviews label Jun 23, 2025
@sowjanyakch sowjanyakch marked this pull request as ready for review June 23, 2025 17:38
Copy link
Copy Markdown
Collaborator

@mahibi mahibi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For me, it crashes with

Process: com.nextcloud.talk2, PID: 27437
java.lang.NullPointerException
	at com.nextcloud.talk.chat.MessageInputFragment.setEditUI$lambda$30(MessageInputFragment.kt:969)
	at com.nextcloud.talk.chat.MessageInputFragment.$r8$lambda$Utfaqgv7v4a77nbj_AuOP9YeTyQ(Unknown Source:0)
	at com.nextcloud.talk.chat.MessageInputFragment$$ExternalSyntheticLambda10.onClick(D8$$SyntheticClass:0)
	at android.view.View.performClick(View.java:8052)
	at android.view.View.performClickInternal(View.java:8029)
	at android.view.View.-$$Nest$mperformClickInternal(Unknown Source:0)
	at android.view.View$PerformClick.run(View.java:31419)
	at android.os.Handler.handleCallback(Handler.java:1001)
	at android.os.Handler.dispatchMessage(Handler.java:105)
	at android.os.Looper.loopOnce(Looper.java:268)
	at android.os.Looper.loop(Looper.java:384)
	at android.app.ActivityThread.main(ActivityThread.java:8921)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:580)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:907)

when i mention someone

Comment thread app/src/main/java/com/nextcloud/talk/utils/message/MessageUtils.kt Outdated
@sowjanyakch sowjanyakch force-pushed the edit_messages branch 4 times, most recently from 7dd88e5 to 5821f20 Compare July 1, 2025 11:41
@sowjanyakch sowjanyakch requested a review from mahibi July 1, 2025 11:44
@sowjanyakch sowjanyakch force-pushed the edit_messages branch 2 times, most recently from 817a490 to 6ac929e Compare July 1, 2025 13:42
@sowjanyakch sowjanyakch force-pushed the edit_messages branch 2 times, most recently from 2f81499 to e7972b7 Compare July 7, 2025 07:21
@sowjanyakch sowjanyakch enabled auto-merge July 7, 2025 07:22
Comment thread app/src/main/java/com/nextcloud/talk/chat/MessageInputFragment.kt Outdated
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 8, 2025

Codacy

Lint

TypemasterPR
Warnings9696
Errors00

SpotBugs

CategoryBaseNew
Bad practice66
Correctness1616
Dodgy code6161
Internationalization33
Malicious code vulnerability33
Performance44
Security11
Total9494

@sowjanyakch sowjanyakch merged commit 4667fef into master Jul 8, 2025
15 of 16 checks passed
@sowjanyakch sowjanyakch deleted the edit_messages branch July 8, 2025 08:57
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 8, 2025

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/5072-talk.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 Talk app.

@sowjanyakch
Copy link
Copy Markdown
Contributor Author

/backport to stable-21.1

@AndyScherzinger AndyScherzinger added this to the 22.0.0 milestone Jul 8, 2025
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.

Editing a message with mentioned users replaces them with these curly bracket tags.

4 participants