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

Add Microsoft Teams notifications #224

Merged
merged 2 commits into from
Mar 25, 2021
Merged

Conversation

MIsmail80
Copy link
Contributor

Fixes #222

This adds supporting of Microsoft Teams notifications. Ex:

@finished
    @teams('teams-webhook', 'message = null', 'theme = 'success'', 'options = []')
@endfinished

Screenshot 2021-03-25 032311

Parameters Description:

teams-webhook: Required.
You should create an incoming webhook for the selected channel to receive notifications, check Teams incoming webhooks docs for more information.

message: Optional.
Default value is Envoy default message, but you can overwrite it with your custom message.

theme: Optional.
Default value is success for green color, but you can choose from three additional theme:

  • info for cyan color
  • error for red color
  • warning for orange color

options: Optional.
Teams API has many other attributes to customize your message box like title, summary, and sections...
You can use this options param. as an array to pass additional attributes to Teams API.

src/Compiler.php Outdated
@@ -47,6 +47,7 @@ class Compiler
'Slack',
'Discord',
'Telegram',
'Teams',
Copy link
Member

Choose a reason for hiding this comment

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

Maybe we should rename this (and all other references) to MicrosoftTeams? I realize "Teams" is the app name but it's so generic that it could be confusing.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree with you, I'll working on this.

Copy link
Member

Choose a reason for hiding this comment

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

@MIsmail80 I've converted the PR to draft until you've made these changes. Feel free to mark it as ready again when you've done the changes.

@driesvints driesvints marked this pull request as draft March 25, 2021 13:40
@MIsmail80 MIsmail80 marked this pull request as ready for review March 25, 2021 14:29
@taylorotwell taylorotwell merged commit a66e830 into laravel:2.x Mar 25, 2021
@MIsmail80 MIsmail80 deleted the teams branch March 25, 2021 15:18
@usmanjdn93
Copy link

@taylorotwell @driesvints can any one add this to the docs please...notification

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.

Support Microsoft Teams notifications
4 participants