Chat - Message that are bigger than 10240 characters does not appear without reload

[kavimuru]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Issue was found when executing #6673

Action Performed:

1. Launch the app and login
2. Open two different accounts and start a chat with each other
3. Write any messages that are bigger than 10240 characters

Expected Result:

Verify that the message will appear for the receiver without needing to reload the chat.

Actual Result:

The message will appear for the receiver after reload of the chat.

Workaround:

Unknown

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Version Number: 1.1.20-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Recording.1156.mp4

Issue was found when executing #6673

Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:

View all open jobs on GitHub

[OSBotify]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[MelvinBot]

Triggered auto assignment to @mountiny (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[mountiny]

I think this is quite edge case so it does not have to be a deploy blocker. Gonna remove the label and make this an External issue

[MelvinBot]

Triggered auto assignment to @NicMendonca (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

[mountiny]

cc @danieldoglas as it is related to the issue you worked on: #3797

It should not work on production as the linked PR has been only deployed only to staging so far, but the issue body says it also happens in Staging.

[mountiny]

Sorry @NicMendonca for a ping, upon further investigation, this should not be external yet. This issue is about merged PR not behaving as expected.

[mvtglobally]

Issue not reproducible during KI retests. (First week)

[danieldoglas]

Thanks for tagging me @mountiny .

@mvtglobally , you could not reproduce this issue too? I downloaded the branch and it was working as expected.

[mvtglobally]

@danieldoglas i was able to repro the issue on Desktop app yesterday with the build 1.1.20-2. It seems to be inconsistent and not always repro.
See tread https://expensify.slack.com/archives/C01GTK53T8Q/p1639532748263800

[mountiny]

Assigning Daniel after chatting in DM together.

[danieldoglas]

Ok, so I just downloaded the latest code in Main and tested this locally to check if it was happening.

This video has the production verion (1.18) and the one in Mani (1.21).

Seems to be working as expected.

Screen.Recording.2021-12-15.at.18.32.37.mov

Also, testing staging.new.expensify produced the same results:

Screen.Recording.2021-12-15.at.18.36.20.mov

I'll try to install the specific version you're testing.

[danieldoglas]

@mvtglobally Can you run a quick test for me? Can you logout and login the application again and see if this continues?
I may have found the reason.

[mvtglobally]

@danieldoglas, sorry missed this earlier. I am still able to reproduce this issue after log out.

[danieldoglas]

I see.

Seems like we've deployed version 1.1.22 version in prod now. I just tested the Lorem Ipsum generated text and it did work. I've tried from 10 to 50 thousand bytes.

For some reason I cant simulate what's happening to you. Are you trying a specific text, or any text with 10240+ chars are not working for you?

if it's a specific test, can you provide it so I can see if it's something specific?

[danieldoglas]

@mvtglobally please tell me if this continues happening. I'm changing it to monthly for now.