Skip to content

Conversation

@jessiezhu-m
Copy link

@jessiezhu-m jessiezhu-m commented Oct 30, 2025

Description

Update Az.Dashboard API version from 2022-08-01 to 2025-08-01. It includes dashboard and private endpoint operations,

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

Copilot AI review requested due to automatic review settings October 30, 2025 10:07
@azure-client-tools-bot-prd
Copy link

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Az.Dashboard module to API version 2025-08-01, adding support for dashboard operations including managed dashboards, managed private endpoints, and integration fabrics. The changes include new cmdlets, updated help documentation, test files, and configuration updates.

Reviewed Changes

Copilot reviewed 70 out of 70 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
src/Dashboard/Dashboard/ChangeLog.md Added version 0.3.0 entry documenting swagger update to 2025-08-01
src/Dashboard/Dashboard/Az.Dashboard.psd1 Updated module metadata, version, required modules, and exported functions
src/Dashboard/Dashboard.sln Updated project GUID references
src/Dashboard/Dashboard/help/*.md Added help documentation for 12 new cmdlets
src/Dashboard/Dashboard.Autorest/test/*.Tests.ps1 Added test files for new cmdlets (all skipped)
src/Dashboard/Dashboard.Autorest/examples/*.md Added placeholder examples for new cmdlets
src/Dashboard/Dashboard.Autorest/docs/*.md Added internal documentation for new cmdlets
src/Dashboard/Dashboard.Autorest/UX/*.json Added UX metadata for new resource types
src/Dashboard/Dashboard.Autorest/README.md Updated commit hash and API version
src/Dashboard/Dashboard.Autorest/generate-info.json Updated generate ID
Comments suppressed due to low confidence (1)

src/Dashboard/Dashboard/Az.Dashboard.psd1:1

  • The required Az.Accounts module version has been updated from 4.2.0 to 5.3.0. Ensure this version dependency is documented in the ChangeLog.md and that users are aware of this breaking change, as they will need to update Az.Accounts.

``` yaml
commit: 02ed6d4aac29881364f8698b4fdac9c76cd0f538
commit: a6ef905ba314503e8cfac82d63a2e790fae7991b
Copy link

Copilot AI Oct 30, 2025

Choose a reason for hiding this comment

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

The commit hash should reference a specific commit from the Azure REST API specs repository to ensure reproducible builds. Verify that this commit hash exists and contains the expected 2025-08-01 API specification.

Copilot generated this review using guidance from repository custom instructions.
```
### -SmtpFromAddress
Address used when sending out emailshttps://pkg.go.dev/net/mail#Address
Copy link

Copilot AI Oct 30, 2025

Choose a reason for hiding this comment

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

Missing space between 'emails' and the URL. Should be 'Address used when sending out emails https://pkg.go.dev/net/mail#Address'.

Copilot uses AI. Check for mistakes.
### -SmtpFromName
Name to be used when sending out emails.
Default is "Azure Managed Grafana Notification"https://pkg.go.dev/net/mail#Address
Copy link

Copilot AI Oct 30, 2025

Choose a reason for hiding this comment

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

Missing space between the quoted string and the URL. Should be 'Default is "Azure Managed Grafana Notification" https://pkg.go.dev/net/mail#Address'.

Copilot uses AI. Check for mistakes.
### -SmtpSkipVerify
Verify SSL for SMTP server.
Default is falsehttps://pkg.go.dev/crypto/tls#Config
Copy link

Copilot AI Oct 30, 2025

Choose a reason for hiding this comment

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

Missing space between 'false' and the URL. Should be 'Default is false https://pkg.go.dev/crypto/tls#Config'.

Copilot uses AI. Check for mistakes.
```
### -SmtpStartTlsPolicy
The StartTLSPolicy setting of the SMTP configurationhttps://pkg.go.dev/github.com/go-mail/mail#StartTLSPolicy
Copy link

Copilot AI Oct 30, 2025

Choose a reason for hiding this comment

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

Missing space between 'configuration' and the URL. Should be 'The StartTLSPolicy setting of the SMTP configuration https://pkg.go.dev/github.com/go-mail/mail#StartTLSPolicy'.

Copilot uses AI. Check for mistakes.
@dolauli dolauli self-assigned this Oct 31, 2025
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