-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
[Az.WebSites] Remove parsing for SnapshotTime #23236
[Az.WebSites] Remove parsing for SnapshotTime #23236
Conversation
️✔️Az.Accounts
️✔️Az.Compute
️✔️Az.KeyVault
️✔️Az.ManagedServiceIdentity
️✔️Az.Monitor
️✔️Az.Network
️✔️Az.OperationalInsights
️✔️Az.PrivateDns
️✔️Az.Sql
️✔️Az.Storage
️✔️Az.Websites
|
@VeryEarly Kindly review. |
please regenerate help markdown for those cmdlets with syntax changes: |
This PR was labeled "needs-revision" because it has unresolved review comments or CI failures. |
@vkarasala, I closed this PR since it's not updated for months. Feel free to reopen it or create a new one when it's ready. Thanks. |
Description
The changes are to remove DateTime conversion for SnapshotTime. The conversion trims the UTC time and sends incorrect date to backend.
Checklist
CONTRIBUTING.md
and reviewed the following information:generation
branch.ChangeLog.md
file(s) appropriatelyChangeLog.md
file can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
## Upcoming Release
header in the past tense. Add changelog in description section if PR goes intogeneration
branch.ChangeLog.md
if no new release is required, such as fixing test case only.