Skip to content

Conversation

albertodonato
Copy link
Contributor

@albertodonato albertodonato commented Aug 27, 2025

ENG-5591

what

add stacklet_configuration_profile_teams resource

why

support configuring Microsoft Teams integration

testing

sandbox testing, added acceptance tests

docs

updated here

@albertodonato albertodonato requested a review from a team as a code owner August 27, 2025 08:33
[ENG-5591](https://stacklet.atlassian.net/browse/ENG-5591)

### what

add `stacklet_configuration_profile_teams` resource

### why

support configuring Microsoft Teams integration

### testing

sandbox testing, added acceptance tests

### docs

updated here
@albertodonato albertodonato force-pushed the ack/release-configuration-profile-resource-teams branch from 3be3a64 to ea1f98d Compare August 27, 2025 08:58
Copy link
Contributor

@fwereade fwereade left a comment

Choose a reason for hiding this comment

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

Just one suggestion; nicely done, thanks. LGTM.

return
}

names := models.ListItemsIdentifiers(state.Webhooks, "name")
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe an explicit comment about how we're restoring the input ordering here?

(…and maybe this whole repeated block deserves a name of its own for that comment to hang off?)

Copy link
Contributor

@fwereade fwereade left a comment

Choose a reason for hiding this comment

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

Nice, thanks, LGTM.

@albertodonato albertodonato merged commit 77c4d7d into main Aug 27, 2025
6 checks passed
@albertodonato albertodonato deleted the ack/release-configuration-profile-resource-teams branch August 27, 2025 09:50
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