-
Notifications
You must be signed in to change notification settings - Fork 104
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
Sync with plugin-starter-template #91
Conversation
@@ -78,7 +78,7 @@ | |||
"display_name": "Secret Message", | |||
"type": "custom", | |||
"help_text": "The message posted by the demo plugin when the secret phrase is detected.", | |||
"default": "Yay! The random secret string was posted!\n\nGo to the settings page for this plugin in the system console to generate a new random secret." | |||
"default": "Yay! The random secret string was posted! Go to the settings page for this plugin in the system console to generate a new random secret." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to change this because \n
did play nicely with the js world.
Codecov Report
@@ Coverage Diff @@
## master #91 +/- ##
==========================================
- Coverage 17.47% 17.10% -0.37%
==========================================
Files 13 14 +1
Lines 1179 1175 -4
==========================================
- Hits 206 201 -5
Misses 934 934
- Partials 39 40 +1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume most of this code is a straight copy from starter-template
but found one possible weird merge issue.
Let's wait for mattermost/mattermost-plugin-starter-template#89 |
@DHaussermann Would you please give this PR a smoke test? |
Let's wait and cut |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and passed
- No build or deployment issues
- Tested all UI events trigger
- Tested webhook events
- Other brief regression testing
No issues found.
LGTM!
Sorry for the long delay on this PR.
Summary
Downstream changes from https://github.com/mattermost/mattermost-plugin-starter-template