Skip to content

Teams_Alarms_example#2

Merged
cstoeg merged 13 commits intomainfrom
add-teams_alarms-example
Dec 2, 2024
Merged

Teams_Alarms_example#2
cstoeg merged 13 commits intomainfrom
add-teams_alarms-example

Conversation

@cstoeg
Copy link
Collaborator

@cstoeg cstoeg commented Sep 13, 2024

Teams example provided by Martin Kumhera, changed to new folder structure and updated readme by jsteiner

@cstoeg cstoeg requested a review from dhoegerlETM September 13, 2024 08:25
Copy link
Contributor

@dhoegerlETM dhoegerlETM left a comment

Choose a reason for hiding this comment

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

We should only provide English text and screenshots for international customers.

@cstoeg cstoeg requested a review from dhoegerlETM September 20, 2024 07:07
@cstoeg
Copy link
Collaborator Author

cstoeg commented Sep 20, 2024

We should only provide English text and screenshots for international customers.

changed to english readme + pictures

Copy link
Contributor

Choose a reason for hiding this comment

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

As we renamed Node.js Manager to JavaScript Manager => shouldn't we have it consistently here or don't care for the example?

Copy link
Contributor

Choose a reason for hiding this comment

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

Same Node.js Manager rename topic here!

@cstoeg cstoeg merged commit 68ee397 into main Dec 2, 2024
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.

2 participants