RocketChat / Rocket.Chat.ReactNative

Rocket.Chat mobile clients
https://rocket.chat
MIT License
1.99k stars 1.16k forks source link

Regression around Shared Links (invalid tmid) #5695

Closed alanedwardes closed 3 months ago

alanedwardes commented 4 months ago

Description:

When using the "share" feature from an app (or website via Safari) on iOS, the message does not display in Rocket Chat. This is because there is an empty string tmid field in the database - if that is manually removed, the message appears.

Environment Information:

Steps to reproduce:

  1. Open Safari, or an app which enables sharing of links (such as a news app)
  2. Tap the "share" button (this is located in the middle of the bottom button row in Safari) image
  3. Tap Rocket Chat, and select a channel

Expected behavior:

The link is shared as expected

Actual behavior:

The message is not displayed, because it is written to the database with an empty string tmid field. If this field is manually removed, the message appears.

Additional context:

This started happening with version 4.49.0 of the app, and I do see an empty string being set for tmid as part of https://github.com/RocketChat/Rocket.Chat.ReactNative/pull/5670

paulchen commented 3 months ago

I can confirm this erroneous behaviour on Android for version 4.49.0. When downgrading to 4.48.0, everything works as expected. As described above, the problem is the empty tmid field. After removing it, the message is displayed as expected.

@alanedwardes Thanks for investigating into this and for reporting it!

diegolmello commented 3 months ago

Thanks for reporting. I was able to reproduce it.