Skip to content

[FIX] Messages attributed to wrong user#4598

Merged
reinaldonetof merged 2 commits into
developfrom
fix.messages-in-sequence
Oct 31, 2022
Merged

[FIX] Messages attributed to wrong user#4598
reinaldonetof merged 2 commits into
developfrom
fix.messages-in-sequence

Conversation

@reinaldonetof

@reinaldonetof reinaldonetof commented Oct 7, 2022

Copy link
Copy Markdown
Contributor

Proposed changes

Under certain circumstances, messages sent by the app user are shown as originating from another user.

Issue(s)

Closes #4559

How to test or reproduce

You need two different users.

  • In a desktop browser, open Rocket.Chat and login with one of the users. Navigate to an arbitrary channel.

  • Open the Rocket.Chat app and login with the other user. Navigate to the same channel as in step 1.

  • Submit a message from the app.

  • Prepare another message in the app, but do not submit it right now.

  • Send a message from the desktop browser.

  • Submit the message prepared in step 4. It is important to submit the message before the message from step 5 is displayed in the app.

  • Submit another message from the app.

Screenshots

Before

Screen.Recording.2022-10-07.at.11.52.04.mov

After

Screen.Recording.2022-10-07.at.11.46.47.mov

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves a current function)
  • New feature (non-breaking change which adds functionality)
  • Documentation update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • I have added necessary documentation (if applicable)
  • Any dependent changes have been merged and published in downstream modules

Further comments

Jira: TC-160

@reinaldonetof reinaldonetof self-assigned this Oct 7, 2022
@reinaldonetof reinaldonetof merged commit 816f43b into develop Oct 31, 2022
@reinaldonetof reinaldonetof deleted the fix.messages-in-sequence branch October 31, 2022 17:02
ivnxyz pushed a commit to NextiaDev/Rocket.Chat.ReactNative that referenced this pull request May 26, 2023
ivnxyz pushed a commit to NextiaDev/Rocket.Chat.ReactNative that referenced this pull request May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Messages attributed to wrong user

2 participants