Skip to content

Comments

Notify if user already exists in room#14142

Closed
gsunit wants to merge 6 commits intoRocketChat:developfrom
gsunit:notify-user-already-exists
Closed

Notify if user already exists in room#14142
gsunit wants to merge 6 commits intoRocketChat:developfrom
gsunit:notify-user-already-exists

Conversation

@gsunit
Copy link
Contributor

@gsunit gsunit commented Apr 15, 2019

A solution to issue #13879. With this we can issue an elegant little notification, which is only visible to the user if a the user invites an already existing member to a room.

Rocket Chat notification

@ggazzo ggazzo added this to the 1.1.0 milestone Apr 17, 2019
});
}
addUserToRoom(data.rid, newUser, user);
// addUserToRoom(data.rid, newUser, user);
Copy link
Contributor

Choose a reason for hiding this comment

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

@gsunit Remove this commented code, please.

Copy link
Contributor Author

@gsunit gsunit May 17, 2019

Choose a reason for hiding this comment

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

I see that you have opened a new PR. Thanks! Should I close this one?

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.

3 participants