Skip to content

Conversation

@alansikora
Copy link
Contributor

@alansikora alansikora commented Mar 30, 2020

Closes #16878
Closes #16861
Closes #16879

Copy link
Contributor

@ndroo ndroo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like it addresses a bug I've been experiencing related to room invites not working. I'll test and reply back.

Copy link
Contributor

@ndroo ndroo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This corrected the issue i had around federated room invites not working. Thank you!

@1rV1N-git
Copy link
Contributor

1rV1N-git commented Mar 31, 2020

Hello
I already write about it
#16878
#16861
#16879
this issue after this commit
7df3bb7

P.S. we have the other issue
#16879

@alansikora
Copy link
Contributor Author

alansikora commented Mar 31, 2020

hey @1rV1N-git! Thanks for your contributions!

I will check #16879 and add a fix on this PR as well.

EDIT: is is fixed, I just added the event generation and handling.

@alansikora alansikora changed the title Simple federation fixes Small federation fixes Mar 31, 2020
@alansikora alansikora marked this pull request as ready for review March 31, 2020 12:25
@ndroo
Copy link
Contributor

ndroo commented Mar 31, 2020

This is likely unrelated but I've found that when I use the mobile client the user being invited into a room bas a weird experience. The room doesnt appear in their list of channels until they click the notification, which unfortunately means if they dont action the notification they never actually make it into the room.

I'm going to verify this today, i was looking at it around midnight after a long day and want to document the actual issue clearly.

@ndroo
Copy link
Contributor

ndroo commented Apr 3, 2020

I merged all these changes into our test build, they all work really well. Thanks for contributing them. I'll let someone from RC review this, but just wanted to drop a thanks :)

I'm going to look at contributing some other features i.e. having /invite work even when the user has not yet been included from a federation perspective

@sampaiodiego sampaiodiego changed the title Small federation fixes [NEW] Federation event for when users left rooms Apr 21, 2020
@sampaiodiego sampaiodiego merged commit f1d1951 into develop Apr 21, 2020
@sampaiodiego sampaiodiego deleted the federation-fixes branch April 21, 2020 08:01
gabriellsh added a commit that referenced this pull request Apr 22, 2020
…users_and_rooms

* 'develop' of github.com:RocketChat/Rocket.Chat: (29 commits)
  [FIX] 2FA not showing codes for Spanish translation (#17378)
  [NEW] [ENTERPRISE] Restrict the permissions configuration for guest users  (#17333)
  [NEW] Federation event for when users left rooms (#17091)
  [FIX] CSV Importer fails when there are no users to import (#16790)
  Import slack's mpims as direct rooms instead of private groups (#17206)
  [FIX] SAML Idp Initiated Logout Error (#17324)
  [NEW] Better Push and Email Notification logic (#17357)
  [NEW] Error page when browser is not supported (#17372)
  [NEW] [ENTERPRISE] Omnichannel queue priorities (#17141)
  [IMPROVE] Change the SAML metadata order to conform to XSD specification (#15488)
  [IMPROVE] Filter markdown in notifications (#9995)
  [IMPROVE] User gets UI feedback when message is pinned or unpinned (#16056)
  Remove set as alias setting (#16343)
  [IMPROVE] Add `file-title` and `file-desc` as new filter tag options on message search (#16858)
  [NEW]  Add ability to set tags in the Omnichannel room closing dialog (#17254)
  [FIX] Show active admin and user account menu item (#17047)
  [NEW] [ENTERPRISE] Allows to set a group of departments accepted for forwarding chats (#17335)
  [FIX] Prevent user from getting stuck on login, if there is some bad fname (#17331)
  [FIX] Remove properties from users.info response (#17238)
  Bump version to 3.1.1
  ...
@sampaiodiego sampaiodiego mentioned this pull request Apr 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When a user left a federated room. The federated server doesn't know it.

5 participants