Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make it clear MNPS is required #1760

Merged
merged 1 commit into from
Jun 19, 2018
Merged

Make it clear MNPS is required #1760

merged 1 commit into from
Jun 19, 2018

Conversation

it33
Copy link

@it33 it33 commented Jun 12, 2018

Please make sure you've read the pull request section of our code contribution guidelines.

When filling in a section please remove the help text and the above text.

Summary

[A brief description of what this pull request does.]

Ticket Link

[Please link the GitHub issue or Jira ticket this PR addresses.]

Checklist

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

  • Added or updated unit tests (required for all new features)
  • All new/modified APIs include changes to mattermost-redux (please link)
  • Has UI changes
  • Includes text changes and localization file updates

Device Information

This PR was tested on: [Device name(s), OS version(s)]

Screenshots

[If the PR includes UI changes, include screenshots (for both iOS and Android if possible).]

@enahum enahum added the 4: Reviews Complete All reviewers have approved the pull request label Jun 12, 2018
@enahum enahum merged commit 79f241a into master Jun 19, 2018
@enahum enahum deleted the it33-patch-1 branch June 19, 2018 23:14
@lindy65 lindy65 added the Tests/Not Needed Does not require tests label Apr 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request Tests/Not Needed Does not require tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants