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

MM-15068: Impossible to join directly to team by url if is not in the first 200 visible teams #2646

Merged
merged 1 commit into from
Apr 12, 2019

Conversation

jespino
Copy link
Member

@jespino jespino commented Apr 11, 2019

Summary

Before this commit to try to join a team we get the list of teams (limited to
200), and match the correct team using the name. Now we get directly the team
by name, this allow us to join any team, not only that teams that are in the
list of the first 200 teams.

Ticket Link

MM-15068

@jespino jespino requested review from sudheerDev and migbot April 11, 2019 15:56
@jespino jespino added the 2: Dev Review Requires review by a core commiter label Apr 11, 2019
@jespino jespino force-pushed the fix-join-team-by-url branch 2 times, most recently from 8744a3a to 5f46f3b Compare April 11, 2019 16:45
@jespino jespino added the CherryPick/Approved Meant for the quality or patch release tracked in the milestone label Apr 11, 2019
@jespino jespino added this to the v5.10.0 milestone Apr 11, 2019
@jespino jespino closed this Apr 11, 2019
@jespino jespino reopened this Apr 11, 2019
@jespino jespino force-pushed the fix-join-team-by-url branch from 5f46f3b to 96ec406 Compare April 11, 2019 20:03
@jespino jespino force-pushed the fix-join-team-by-url branch from 96ec406 to 0130f47 Compare April 11, 2019 20:13
@migbot migbot merged commit cc09640 into mattermost:master Apr 12, 2019
migbot pushed a commit that referenced this pull request Apr 12, 2019
@migbot migbot added CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone and removed 2: Dev Review Requires review by a core commiter CherryPick/Approved Meant for the quality or patch release tracked in the milestone labels Apr 12, 2019
@jespino jespino deleted the fix-join-team-by-url branch April 12, 2019 17:58
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Apr 12, 2019
@lindy65 lindy65 added the Tests/Not Needed Does not require new release tests label Apr 13, 2019
@lindalumitchell lindalumitchell added Tests/Done Release tests have been written and removed Tests/Not Needed Does not require new release tests labels May 3, 2019
TranMacTien pushed a commit to Designveloper/mattermost-webapp that referenced this pull request Jun 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Changelog/Not Needed Does not require a changelog entry CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone Docs/Not Needed Does not require documentation Tests/Done Release tests have been written
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants