Skip to content

fix discussions: remove restriction for editing room info, server side#14039

Merged
ggazzo merged 3 commits intoRocketChat:developfrom
assistify:fix/#14027-edit-discussion-rooms
May 6, 2019
Merged

fix discussions: remove restriction for editing room info, server side#14039
ggazzo merged 3 commits intoRocketChat:developfrom
assistify:fix/#14027-edit-discussion-rooms

Conversation

@mrsimpson
Copy link
Contributor

Closes #14027

@ggazzo Obviously, you restricted editing the discussion's room info not by accident ;) I just tested removing the restrictions and can't see a reason why one should not edit it. Also changing the name was stable: The navigation was working after that as well.

Only deleting the discussion will lead to an orphaned "discussion-created-message" in the parent channel. This will lead to a "not allowed" message just like a navigation to a discussion you are not a member of.

@mrsimpson mrsimpson requested a review from ggazzo April 8, 2019 12:05
@mrsimpson mrsimpson self-assigned this Apr 8, 2019
@ggazzo
Copy link
Member

ggazzo commented Apr 8, 2019

@mrsimpson about the orphan message: #14018

Copy link
Member

@ggazzo ggazzo left a comment

Choose a reason for hiding this comment

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

we are using topic field to set the parent name, so sure we can enable edition but we should block topic changes...

look Rocket.Chat/app/channel-settings/client/views/channelSettings.html and Rocket.Chat/app/channel-settings/client/views/channelSettings.js

and use something like roomTypes.getConfig(room.t).allowRoomSettingChange(room, RoomSettingsEnum.TOPIC)) {

@ggazzo ggazzo added this to the 1.1.0 milestone Apr 8, 2019
@mrsimpson
Copy link
Contributor Author

@ggazzo in our practice it has shown that if somebody wants to change the topic (“re-assign it to some other topic”) there’s a good reason for that. And since the topic property is not technically needed for the relationship, I don’t see a reason why to block editing.

Sent with GitHawk

@ggazzo ggazzo changed the title [FIX] Discussions: remove restriction for editing room info fix discussions: remove restriction for editing room info, server side May 6, 2019
@ggazzo ggazzo merged commit fc27d11 into RocketChat:develop May 6, 2019
@sampaiodiego sampaiodiego mentioned this pull request May 28, 2019
@mrsimpson mrsimpson deleted the fix/#14027-edit-discussion-rooms branch December 7, 2020 09:09
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.

Discussions: Can't edit the room info

3 participants

Comments