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

Upgrade appservice package #3254

Merged
merged 6 commits into from
Jul 21, 2022
Merged

Upgrade appservice package #3254

merged 6 commits into from
Jul 21, 2022

Conversation

nturinski
Copy link
Member

@nturinski nturinski commented Jul 21, 2022

Fixes #2835

Need to actually release the new appservice package and install package-lock.json.

This will pick up the new App Insights 2.1 stuff along with the deployment ID tracking.

@nturinski nturinski requested a review from a team as a code owner July 21, 2022 20:36
alexweininger
alexweininger previously approved these changes Jul 21, 2022
package.json Outdated Show resolved Hide resolved
nturinski and others added 2 commits July 21, 2022 13:43
Co-authored-by: Brandon Waterloo [MSFT] <36966225+bwateratmsft@users.noreply.github.com>
@nturinski nturinski merged commit 8524675 into main Jul 21, 2022
@nturinski nturinski deleted the nat/ai branch July 21, 2022 21:47
@nturinski
Copy link
Member Author

Tests are breaking because some triggers no longer have the (preview) suffix after they've updated the templates. I'll fix this in a separate PR.

@microsoft microsoft locked and limited conversation to collaborators Sep 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

App Insights APM 2.1 Enablement and Representation
3 participants