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

fix power automate opensearch integration #1519

Merged
merged 3 commits into from
Aug 18, 2024
Merged

Conversation

jertel
Copy link
Owner

@jertel jertel commented Aug 17, 2024

Description

Corrects an Opensearch integration mistake with the new Microst Power Automate alerter.

Checklist

  • I have reviewed the contributing guidelines.
  • I have included unit tests for my changes or additions.
  • I have successfully run make test-docker with my changes.
  • I have manually tested all relevant modes of the change in this PR.
  • [N/A] I have updated the documentation.
  • I have updated the changelog.

Questions or Comments

@marssilva
Copy link
Contributor

Hello @jertel

Thanks for the adjustment. All that was missing was the documentation.

`
ms_power_automate_opensearch_discover_color: By default, the alert will be published with the ``default`` type blue if not specified. If set to ``positive``, action is displayed with a positive style (typically the button becomes accent color), If set to ``destructive``, Action is displayed with a destructive style (typically the button becomes red)

`

@nsano-rururu nsano-rururu merged commit a640ece into master Aug 18, 2024
1 check passed
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.

3 participants