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
Regression: broken message exhibition when edited #28180
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## develop #28180 +/- ##
===========================================
+ Coverage 44.81% 44.86% +0.04%
===========================================
Files 766 766
Lines 14947 14947
Branches 2087 2087
===========================================
+ Hits 6699 6706 +7
+ Misses 7952 7945 -7
Partials 296 296
Flags with carried forward coverage won't be shown. Click here to find out more. |
tassoevan
force-pushed
the
regression/edited-message-bug
branch
from
February 27, 2023 13:59
baf75a4
to
eda4718
Compare
sampaiodiego
previously requested changes
Feb 27, 2023
apps/meteor/app/federation-v2/server/infrastructure/rocket-chat/hooks/index.ts
Outdated
Show resolved
Hide resolved
ggazzo
previously approved these changes
Feb 28, 2023
dionisio-bot
bot
added
stat: ready to merge
PR tested and approved waiting for merge
and removed
stat: needs QA
labels
Mar 1, 2023
dionisio-bot
bot
added
stat: ready to merge
PR tested and approved waiting for merge
and removed
stat: ready to merge
PR tested and approved waiting for merge
labels
Mar 1, 2023
dionisio-bot
bot
added
stat: ready to merge
PR tested and approved waiting for merge
and removed
stat: ready to merge
PR tested and approved waiting for merge
labels
Mar 1, 2023
dionisio-bot
bot
added
stat: ready to merge
PR tested and approved waiting for merge
and removed
stat: ready to merge
PR tested and approved waiting for merge
labels
Mar 1, 2023
sampaiodiego
approved these changes
Mar 1, 2023
ggazzo
approved these changes
Mar 1, 2023
dionisio-bot
bot
added
stat: ready to merge
PR tested and approved waiting for merge
and removed
stat: ready to merge
PR tested and approved waiting for merge
labels
Mar 1, 2023
dionisio-bot
bot
added
stat: ready to merge
PR tested and approved waiting for merge
and removed
stat: ready to merge
PR tested and approved waiting for merge
labels
Mar 2, 2023
gabriellsh
added a commit
that referenced
this pull request
Mar 6, 2023
…search-nonlatin-rooms * 'develop' of github.com:RocketChat/Rocket.Chat: (41 commits) Regression: Apps provider not handling apps lists correctly (#28264) Regression: Fix intermitent apps engine startup (#28254) Regression: Client is not loading new settings from server (#28248) Regression: Adjusted departments upsell modal to check for licence and modules (#28246) Regression: message-parse version update (#28245) [FIX] Notify apps engine event through ms api (#28169) Chore: Add extra stats for omnichannel video calling (#28211) Regression: Adjusted components using PaginatedMultiSelected to be compatible with new Fuselage version (#28233) Regression: Change text of buttons tooltips on custom content homepage block (#28094) Regression: run clientMessageRecieved hooks on thread main message (#27962) Regression: Jump to message not working (#28163) Regression: Hide price and category filters when inside of private apps page (#28188) [BREAK] Remove support to deprecated typing event (#28234) Regression: Fix app status filter not working in private apps page (#28198) Regression: UiKit message surface styles (#28213) [FIX] Search list missing key parameter (#28230) Regression: broken message exhibition when edited (#28180) Regression: EnableUnlimitedApps button action (#28221) Regression: Update Composer Formatters Translation Keys (#28228) Regression: EmojiPicker missing shadow on dark mode (#28236) ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes (including videos or screenshots)
Sometimes a message will have the fields
editedBy
andeditedAt
present but set tonull
. This PR handle this situation.Issue(s)
Steps to test or reproduce
Further comments