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

MM-9756: Don't show flag in menu on system messages. #940

Merged
merged 1 commit into from
Mar 13, 2018

Conversation

grundleborg
Copy link
Contributor

Summary

Don't show the "Flag" option in dot menu for System Messages.

Ticket Link

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

@grundleborg grundleborg added the 2: Dev Review Requires review by a core commiter label Mar 13, 2018
@grundleborg grundleborg added this to the v4.9.0 milestone Mar 13, 2018
Copy link
Member

@saturninoabril saturninoabril left a comment

Choose a reason for hiding this comment

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

LGTM, just need to update test snapshot

Copy link
Contributor

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

lgtm. same comment as @saturninoabril

@jwilander jwilander added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Mar 13, 2018
@jwilander jwilander merged commit dae1d28 into mattermost:master Mar 13, 2018
JasperVanEsveld added a commit to delftswa2018/mattermost-webapp that referenced this pull request Mar 16, 2018
* upstream/master: (60 commits)
  Migrate admin console to a new declarative way (plugins schema based) (mattermost#905)
  MM-9776: Don't sync search highlighting with search terms. (mattermost#942)
  [MM-9783] Fix file upload removal by setting intl as context types instead of props to easily access component via ref (mattermost#961)
  Updating help text on custom emoji creation page (mattermost#959)
  Update mattermost-redux to latest webapp-4.8 (mattermost#960)
  MM-9845: Moves event handlers back to img tag. (mattermost#956)
  Fix blank channel header when leaving team (mattermost#958)
  If experimental auth change setting isn't set, default to old behavior (mattermost#954)
  fix search bar focus after clicking pinnned or flagged post button (mattermost#953)
  MM-9825: Show again compliance reports (mattermost#952)
  translations PR 20180314 (mattermost#950)
  MM-9819 Kept cursor in the correct place on IE11/Windows 7 when typing (mattermost#945)
  Fix PDF previews on Firefox (mattermost#944)
  Filter out current user when adding member to GM (mattermost#946)
  MM-9743 Fix RHS focus when deleting and editing posts (mattermost#947)
  MM-9756: Don't show flag in menu on system messages. (mattermost#940)
  fix JS errors on draft - post or comment (mattermost#920)
  4.8 fixes (mattermost#934) MM-9789 and MM-9786
  MM-9773 Fixed permalinks on IE11 (mattermost#936)
  MM-9780: Show name in "removed from team" system message. (mattermost#939)
  ...
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Mar 16, 2018
@lindalumitchell lindalumitchell added the Tests/Done Release tests have been written label Mar 23, 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/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation Tests/Done Release tests have been written
Projects
None yet
6 participants