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

MM-8701 Get client config after logging in #1363

Merged
merged 1 commit into from
Jun 20, 2018
Merged

MM-8701 Get client config after logging in #1363

merged 1 commit into from
Jun 20, 2018

Conversation

hmhealey
Copy link
Member

This is in preparation for an eventual change to stop the server from sending out the full client config to users who aren't logged in

Ticket Link

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

@hmhealey hmhealey added the 2: Dev Review Requires review by a core commiter label Jun 20, 2018
@hmhealey hmhealey added this to the v5.1.0 milestone Jun 20, 2018
@enahum
Copy link
Contributor

enahum commented Jun 20, 2018

@hmhealey can you link the redux PR? I do not know what was changed there

Copy link
Contributor

@enahum enahum left a comment

Choose a reason for hiding this comment

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

after looking at the redux PR this looks good

@sudheerDev sudheerDev added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Jun 20, 2018
@sudheerDev sudheerDev merged commit d405845 into master Jun 20, 2018
@sudheerDev sudheerDev deleted the mm8701 branch June 20, 2018 19:17
masonicboom pushed a commit to masonicboom/mattermost-webapp that referenced this pull request Jun 22, 2018
@amyblais amyblais added Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation labels Jun 25, 2018
@lindalumitchell lindalumitchell added the Tests/Not Needed Does not require new release tests label Jun 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4: Reviews Complete All reviewers have approved the pull request Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation Tests/Not Needed Does not require new release tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants