Mattermost-server: [Help Wanted] Post a System Message when channel is renamed

[it33]

When someone changes the name of a channel or private group, post a system message indicating the change with:

[USERNAME] updated the channel name from: [OLD_NAME] to: [NEW_NAME]

Here is an example from existing functionality changing the header of a channel:

[jasonblais]

Closing, duplicate of https://github.com/mattermost/platform/issues/4333

Work in progress PR also submitted that will add this functionality: https://github.com/mattermost/platform/pull/4664

[jasonblais]

Reopening, this was not in the scope of #4333.

If anyone in the community is interested implementing this, let us know!

[fraziern]

I'll give this a try (still getting up to speed on Golang so I may have questions 😄 ).

[yangchen1]

@fraziern awesome! Please let us know if you have any questions, or post in pre-release.

[fraziern]

@yangchen1 @jasonblais I see there is a display_name and a name (aka handle) attribute for each channel. Should there be a message displayed when either is changed, or just the display_name? If both, should there be different messages?

[jasonblais]

Hey @fraziern, an excellent question.

The help wanted issue was originally intended for display_name only, but I would see a lot of use cases where it would be useful to know if the name has changed as well (renaming the handle can brake public links, bookmarks and integrations, for instance).

I've queued this for our UX meeting to discuss.

If we were to create a system message when the name is updated as well, would you prefer a second help wanted issue for it, or just add it to this issue?

[fraziern]

@jasonblais I'm happy to tackle both at once... maybe there will be some code reuse opportunities. 😄

[jasonblais]

Thanks, sounds good @fraziern! I'll let you know after our UX meeting, it's on Wednesday at 8:30am San Francisco time.

[jasonblais]

Hey @fraziern:

Propose adding a system message only when the display_name is changed. The system message is useful in this use case as we want all channel members to be aware that the channel was renamed.

For changing the name, the use case is limited as most people don't need to know what the URL is.

[jasonblais]

Also, Mattermost Holiday Hackfest starts today, if you're interested in participating :)

[cpanato]

can I work on this? @jasonblais
or @fraziern is working on that?

[jasonblais]

I think @fraziern started work on this :)

[fraziern]

Yep should have a PR out this weekend. :)

[tejasbubane]

I guess this can be closed by #4762