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

[FIX] Message payload from __my_messages__ stream #20801

Merged
merged 1 commit into from
Feb 18, 2021

Conversation

ggazzo
Copy link
Member

@ggazzo ggazzo commented Feb 18, 2021

closes #19566
closes #19567
closes #19433

closes RocketChat/Rocket.Chat.js.SDK#121
closes RocketChat/Rocket.Chat.js.SDK#126
closes RocketChat/Rocket.Chat.js.SDK#127

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@ggazzo ggazzo requested a review from sampaiodiego February 18, 2021 01:06
@ggazzo ggazzo added this to the 3.12.0 milestone Feb 18, 2021
@sampaiodiego sampaiodiego changed the title [FIX] Fixed my_message stream [FIX] Message payload of __my_messages__ stream Feb 18, 2021
@sampaiodiego sampaiodiego changed the title [FIX] Message payload of __my_messages__ stream [FIX] Message payload from __my_messages__ stream Feb 18, 2021
@sampaiodiego sampaiodiego merged commit 384a5f8 into develop Feb 18, 2021
@sampaiodiego sampaiodiego deleted the fix/my_messages branch February 18, 2021 13:22
@sampaiodiego sampaiodiego mentioned this pull request Feb 28, 2021
jsbronder added a commit to jsbronder/wcrc that referenced this pull request Mar 4, 2021
Workaround RocketChat/Rocket.Chat#20801.

Which is to say, upstream made this a list by mistake in 3.8.0, still
hasn't fixed it after 8 point releases and no one can actually determine
if its a bug or not because there's no documentation telling you what to
expect from this stream.  But in 3.12.0, it's reverted, so we need to
handle both.
jsundahl added a commit to jsundahl/skyfall-plugin-rocketchat that referenced this pull request Jul 20, 2021
rocketchat is dumb and sends an array of one message since version 3.8.0.
see RocketChat/Rocket.Chat#20801
this is scheduled to be fixed in version 3.12.0
mdbarr pushed a commit to mdbarr/skyfall-plugin-rocketchat that referenced this pull request Jul 20, 2021
rocketchat is dumb and sends an array of one message since version 3.8.0.
see RocketChat/Rocket.Chat#20801
this is scheduled to be fixed in version 3.12.0
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.

Hubot is breaks with rocket: 3.8.0 Bbot isn't working anymore Hubot not compatible with versions >= 3.8.0
2 participants