Skip to content

Fixes GH-542-Moved : Request for Documentation: Add to interactive-messages documentation#3482

Merged
cpoile merged 3 commits intomattermost:masterfrom
M-ZubairAhmed:GH-542-Moved
Mar 29, 2020
Merged

Fixes GH-542-Moved : Request for Documentation: Add to interactive-messages documentation#3482
cpoile merged 3 commits intomattermost:masterfrom
M-ZubairAhmed:GH-542-Moved

Conversation

@M-ZubairAhmed
Copy link
Copy Markdown
Member

Summary

This PR adds the a very important documentation line, which guides user to how to get the selected value of option of post type Message Menus

Ticket Link

Fixes mattermost/mattermost-developer-documentation#542

@hanzei hanzei requested a review from cpoile March 26, 2020 19:56
@hanzei hanzei added the 1: Dev Review Requires review by a core commiter label Mar 26, 2020
@justinegeffen justinegeffen added the 2: Editor Review Requires review by an editor label Mar 27, 2020
Comment thread source/developer/interactive-messages.rst Outdated
Co-Authored-By: Justine Geffen <justinegeffen@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@justinegeffen justinegeffen left a comment

Choose a reason for hiding this comment

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

LGTM! :)

@justinegeffen justinegeffen removed the 2: Editor Review Requires review by an editor label Mar 27, 2020
@M-ZubairAhmed
Copy link
Copy Markdown
Member Author

Opps sorry i re requested review @justinegeffen

@justinegeffen
Copy link
Copy Markdown
Contributor

No problem at all, thank you! It looks good to me. :)

Copy link
Copy Markdown
Member

@cpoile cpoile left a comment

Choose a reason for hiding this comment

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

Thanks @M-ZubairAhmed !

@cpoile cpoile added 3: Reviews Complete All reviewers have approved the pull request and removed 1: Dev Review Requires review by a core commiter labels Mar 29, 2020
@cpoile cpoile merged commit b70bf48 into mattermost:master Mar 29, 2020
@amyblais amyblais removed the 3: Reviews Complete All reviewers have approved the pull request label Mar 30, 2020
justinegeffen pushed a commit that referenced this pull request Apr 6, 2020
* master:
  Added Let's Encrypt clarification  (#2866)
  Update security.rst (#3434)
  Update sg_grammar-spelling-mechanics.rst (#3462)
  Update training.rst (#3485)
  Update release-faq.rst (#3484)
  Documenting how to change the default docker-compose network (#3480)
  Fixes GH-542-Moved : Request for Documentation: Add to interactive-messages documentation (#3482)
  Correct Code Block Formatting (#3477)
  Update mmctl-cli-tool.rst (#3481)
  FAQ on Archived Channels and data retention (#3470)
  Update desktop-changelog.rst (#3473)
  Update desktop-changelog.rst (#3471)
  Added Video to AD/LDAP Group Sync (#3450)
  Hint to `"skip_slack_parsing"` to ignore Slack-compatibility logic (#3427)
  Update bug-severity-guidelines.rst (#3467)
  Moving Push Proxy related entry to developer documentation (#3448)
justinegeffen pushed a commit that referenced this pull request Apr 8, 2020
…/docs into justinegeffen-patch-7

* 'justinegeffen-patch-7' of https://github.com/mattermost/docs: (90 commits)
  Update integrations.rst
  Remove Oracle from Bitnami packages (#3491)
  Revert "Adding NPS rating for docs pages (#3469)" (#3495)
  Add Plugin Labels and Prepackaged Plugin Info (#3395)
  Adding NPS rating for docs pages (#3469)
  Fix broken link in Bot Account Creation > Plugin (#3493)
  Added Let's Encrypt clarification  (#2866)
  Update security.rst (#3434)
  Update sg_grammar-spelling-mechanics.rst (#3462)
  Update training.rst (#3485)
  Update release-faq.rst (#3484)
  Documenting how to change the default docker-compose network (#3480)
  Fixes GH-542-Moved : Request for Documentation: Add to interactive-messages documentation (#3482)
  Correct Code Block Formatting (#3477)
  Update mmctl-cli-tool.rst (#3481)
  FAQ on Archived Channels and data retention (#3470)
  Update desktop-changelog.rst (#3473)
  Update desktop-changelog.rst (#3471)
  Added Video to AD/LDAP Group Sync (#3450)
  Hint to `"skip_slack_parsing"` to ignore Slack-compatibility logic (#3427)
  ...

# Conflicts:
#	source/install/troubleshooting.rst
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Request for Documentation: Add to interactive-messages documentation

5 participants