mattermost-webapp
mattermost-webapp copied to clipboard
MM-47361 : Remove global state & dispatch from logged_in.tsx
Summary
Included viewChannel
in mapDispatchToProps
so that we don't have to call it with state
and dispatch
inside the component
Ticket Link
fixes https://github.com/mattermost/mattermost-server/issues/21248
Related Pull Requests
~- Has server changes (please link here)~ ~- Has mobile changes (please link here)~
NONE
@batebobo: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.
I understand the commands that are listed here
Hello @batebobo,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.
E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.
cc/ @mkraft
This PR has been automatically labelled "stale" because it hasn't had recent activity. A core team member will check in on the status of the PR to help with questions. Thank you for your contribution!
/e2e-test
Successfully triggered e2e testing! https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/259256
A pipeline with the same environment variables is already running. Please cancel it first with /e2e-cancel, or specify different environment variables.