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

Fix bug when ExperimentalPrimaryTeam is set lastUrl does not work #765

Conversation

csduarte
Copy link
Contributor

@csduarte csduarte commented Feb 9, 2018

Summary

There is an issue were last channel is not being stored when you close the window. Every time you start it takes you to Town Square. This is caused by the logic added with config ExperimentalPrimaryTeam. This PR fixes the issue.
#334

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

via @dmeza

@dmeza dmeza force-pushed the FixForLastUrlWhenExperimentalPrimaryTeamSet branch from 0a3629b to 722c66d Compare February 9, 2018 16:32
@jwilander jwilander added the 2: Dev Review Requires review by a core commiter label Feb 9, 2018
@hmhealey hmhealey removed the 2: Dev Review Requires review by a core commiter label Feb 12, 2018
@hmhealey hmhealey merged commit ba93784 into mattermost:master Feb 12, 2018
@lindalumitchell lindalumitchell added the Tests/Not Needed Does not require new release tests label Feb 13, 2018
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Feb 17, 2018
@dmeza dmeza deleted the FixForLastUrlWhenExperimentalPrimaryTeamSet branch December 7, 2018 18:27
hmhealey pushed a commit that referenced this pull request Aug 28, 2020
* Spliting emojis and webhooks permissions

* Keeping the old constants for backward compatibility
hmhealey pushed a commit that referenced this pull request Mar 17, 2021
* Spliting emojis and webhooks permissions

* Keeping the old constants for backward compatibility
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 Docs/Not Needed Does not require documentation Tests/Not Needed Does not require new release tests
Projects
None yet
6 participants