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

Prevent error on long press on the chatbox #2484

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pieer
Copy link
Collaborator

@pieer pieer commented Dec 28, 2024

No description provided.

@pieer pieer requested a review from taoeffect December 28, 2024 12:16
Copy link

cypress bot commented Dec 28, 2024

group-income    Run #3626

Run Properties:  status check passed Passed #3626  •  git commit fc19697304 ℹ️: Merge 3a1f6f15c61a8816c75c8f29cba5300c00733016 into 25c49c96a96d0b6aa6367fa30329...
Project group-income
Branch Review fix/#2461-error-message-on-long-press
Run status status check passed Passed #3626
Run duration 10m 39s
Commit git commit fc19697304 ℹ️: Merge 3a1f6f15c61a8816c75c8f29cba5300c00733016 into 25c49c96a96d0b6aa6367fa30329...
Committer Pierre Schweiger
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 10
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 112
View all changes introduced in this branch ↗︎

@taoeffect
Copy link
Member

@pieer Thanks for sending this in!

Could you post a screenshot or something to show how to reproduce this error on master?

@pieer
Copy link
Collaborator Author

pieer commented Dec 29, 2024

I don't have screenshot but this is related to this issue: #2461
Where I pin point why there was the error you reported @taoeffect: longPress on the message being edited.
The reason for this is that messageAction is only rendered when the component is not in editing mode.

Before this fix we were trying to access that messageAction on longpress even though the message action was not in the DOM

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.

2 participants