-
Notifications
You must be signed in to change notification settings - Fork 19
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
Implement autorespond feature #188
Conversation
Codecov Report
@@ Coverage Diff @@
## master #188 +/- ##
==========================================
- Coverage 23.52% 23.48% -0.05%
==========================================
Files 62 64 +2
Lines 2516 2602 +86
==========================================
+ Hits 592 611 +19
- Misses 1850 1913 +63
- Partials 74 78 +4
Continue to review full report at Codecov.
|
Thank you very much @MatthewDorner ! I think we can safely close the other PR, and continue with this one, in case any other change is needed (I hope not!). |
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.
Approving as previous PR was already approved.
@DHaussermann @mickmister Merging this PR since the original one was already approved by all of us, and was ready to merge. |
This reverts commit b8aff5a.
This reverts commit b8aff5a.
This is a duplicate of my PR which keeps getting broken since the repo was taken private and then back to public: #126. My
mattermost-plugin-mscalendar
repo still shows itself as being forked fromcpurta/mattermost-plugin-msoffice
so it's still broken, and I'm making this PR from my new fork in case you want to just merge this one or if the other breaks again.