Skip to content

Added ExpirationTime field to Role Assignment response DTO for version 2026-07-01-preview#39224

Open
denyspanov-public wants to merge 1 commit into
Azure:mainfrom
denyspanov-public:main
Open

Added ExpirationTime field to Role Assignment response DTO for version 2026-07-01-preview#39224
denyspanov-public wants to merge 1 commit into
Azure:mainfrom
denyspanov-public:main

Conversation

@denyspanov-public

Copy link
Copy Markdown
Member

Added ExpirationTime field to Role Assignment response DTO for version 2026-07-01-preview

Added ExpirationTime field to Role Assignment response DTO for version 2026-07-01-preview

Command used to generate this PR:**(Applicable only to SDK release request PRs)

Checklists

  • Added impacted package name to the issue description
  • Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
  • Added a changelog (if necessary)

Copilot AI review requested due to automatic review settings July 9, 2026 13:48
@github-actions github-actions Bot added Mgmt This issue is related to a management-plane library. mgmt-review-needed labels Jul 9, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 adds a read-only expirationTime property to the RoleAssignment and RoleAssignmentProperties models in the @azure/arm-authorization management SDK, wiring it through both role-assignment property deserializers so the value returned by get/list operations is exposed to consumers. It is described as backing the 2026-07-01-preview api-version.

The change is internally consistent: the read-only field is correctly added to the public API report, excluded from the (input-only) serializers, and documented with a Features Added CHANGELOG entry that matches the current unreleased 10.0.0-beta.4 version. The one concern is functional: the role-assignment operations that produce these models still request api-version=2022-04-01, which does not return expirationTime, so the field will always deserialize to undefined until those operations are regenerated against 2026-07-01-preview.

Changes:

  • Added readonly expirationTime?: string to RoleAssignment and RoleAssignmentProperties, plus the corresponding lines in both property deserializers.
  • Updated the public API report (review/*.api.md) to include the new property on both interfaces.
  • Added a Features Added CHANGELOG entry for the new property.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
sdk/authorization/arm-authorization/src/models/microsoft/roleAssignment/models.ts Adds expirationTime to both interfaces and both deserializers; correctly omitted from serializers since it is read-only.
sdk/authorization/arm-authorization/review/arm-authorization-node.api.md Reflects the new expirationTime property on both public interfaces, keeping the API report in sync.
sdk/authorization/arm-authorization/CHANGELOG.md Documents the new property under Features Added for the unreleased 10.0.0-beta.4.

Comment on lines +39 to +40
/** Time at which the role assignment expires */
readonly expirationTime?: string;
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Mgmt This issue is related to a management-plane library. mgmt-review-needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants