Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

MM-14410: Adds the ability to add and manage groups for a team. #2707

Merged
merged 7 commits into from
May 10, 2019
Merged

MM-14410: Adds the ability to add and manage groups for a team. #2707

merged 7 commits into from
May 10, 2019

Conversation

mkraft
Copy link
Contributor

@mkraft mkraft commented Apr 25, 2019 •
edited
Loading

Summary

Adds the ability to add and manage groups for a team.

Ticket Link

https://mattermost.atlassian.net/browse/MM-14410

Related Pull Requests

Screenshots

Screen Shot 2019-05-02 at 11 10 32 AM

Screen Shot 2019-05-02 at 11 10 48 AM

Screen Shot 2019-05-02 at 11 10 59 AM

@mkraft mkraft added the Work in Progress Not yet ready for review label Apr 25, 2019
@mkraft mkraft force-pushed the MM-14410 branch 13 times, most recently from a05d64c to 52fe81a Compare May 2, 2019 13:44
@mkraft mkraft changed the title MM-14410 MM-14410: Adds the ability to add and manage groups for a team. May 2, 2019
@mkraft mkraft force-pushed the MM-14410 branch 2 times, most recently from 312a794 to 4691d1b Compare May 3, 2019 11:41
@mkraft mkraft requested review from jespino and mgdelacroix May 3, 2019 17:00
@mkraft mkraft added 2: Dev Review Requires review by a core commiter Do Not Merge/Awaiting PR Awaiting another pull request before merging (e.g. server changes) and removed Work in Progress Not yet ready for review labels May 3, 2019
@amyblais amyblais added this to the v5.12.0 milestone May 9, 2019
jespino
jespino reviewed May 9, 2019
View reviewed changes
components/list_modal.jsx Outdated Show resolved Hide resolved
jespino
jespino reviewed May 9, 2019
View reviewed changes
components/list_modal.jsx Outdated Show resolved Hide resolved
jespino
jespino reviewed May 9, 2019
View reviewed changes
components/list_modal.jsx Outdated Show resolved Hide resolved
mgdelacroix
mgdelacroix approved these changes May 10, 2019
View reviewed changes
Copy link
Member

@mgdelacroix mgdelacroix left a comment

Choose a reason for hiding this comment

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

LGTM

@mkraft mkraft merged commit 7d91053 into master May 10, 2019
@mkraft mkraft deleted the MM-14410 branch May 10, 2019 16:00
@mkraft mkraft mentioned this pull request May 10, 2019
Merged
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation Docs/Needed Requires documentation and removed Docs/Not Needed Does not require documentation labels May 21, 2019
@wiersgallak wiersgallak self-assigned this May 21, 2019
@lindalumitchell lindalumitchell added the Tests/Done Release tests have been written label Jun 4, 2019
@amyblais amyblais added Docs/Done Required documentation has been written and removed 2: Dev Review Requires review by a core commiter Docs/Needed Requires documentation labels Jun 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mgdelacroix mgdelacroix mgdelacroix approved these changes

@jespino jespino jespino approved these changes

Assignees

@wiersgallak wiersgallak

Labels
Changelog/Not Needed Does not require a changelog entry Do Not Merge/Awaiting PR Awaiting another pull request before merging (e.g. server changes) Docs/Done Required documentation has been written Tests/Done Release tests have been written
Projects
None yet
Milestone
v5.12.0
6 participants
@mkraft @mgdelacroix @jespino @lindalumitchell @amyblais @wiersgallak