Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Attachment - Text is duplicated when sent message with attachment while offline #9766

Closed
kbecciv opened this issue Jul 7, 2022 · 8 comments
Assignees
Labels
Daily KSv2 Engineering Improvement Item broken or needs improvement.

Comments

@kbecciv
Copy link

kbecciv commented Jul 7, 2022

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Issue was found when executing PR #9350

Action Performed:

  1. Create two accounts and a chat between them in NewDot
  2. Open a new NewDot window in a new Chrome session (do not use incognito mode).
  3. Turn off the internet
  4. Add an attachment to the chat between these two users while there is text in the composer.
  5. Verify the attachment and text appears in rea ltime for the other user.

Expected Result:

The attachment and text appears in real time for the other user.

Actual Result:

Text is duplicated when sent message with attachment while offline

Workaround:

Unknown

Platform:

Where is this issue occurring?

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

Version Number: 1.1.79.17

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers): any

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Bug5639881_20220707_134456.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause -Internal Team

Slack conversation:

View all open jobs on GitHub

@melvin-bot
Copy link

melvin-bot bot commented Jul 7, 2022

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

@MonilBhavsar MonilBhavsar added the Improvement Item broken or needs improvement. label Jul 8, 2022
@MonilBhavsar
Copy link
Contributor

Will look into it but probably should be hold on API and offline doc.

@melvin-bot
Copy link

melvin-bot bot commented Jul 11, 2022

@MonilBhavsar Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@melvin-bot
Copy link

melvin-bot bot commented Jul 13, 2022

@MonilBhavsar Huh... This is 4 days overdue. Who can take care of this?

@MonilBhavsar
Copy link
Contributor

I'll try to reproduce today and provide an update here

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jul 14, 2022
@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

@MonilBhavsar
Copy link
Contributor

Not overdue

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jul 19, 2022
@MonilBhavsar
Copy link
Contributor

Can't seem to reproduce. Feel free to reopen if issue persists and I can take a look

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2 Engineering Improvement Item broken or needs improvement.
Projects
None yet
Development

No branches or pull requests

3 participants