-
Notifications
You must be signed in to change notification settings - Fork 5.6k
Users/tejasma/2025 09 01 #36629
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
Users/tejasma/2025 09 01 #36629
Changes from 1 commit
f9097e6
bb4b61c
31f301a
d39388d
2ad7c75
be9d358
4890bb9
49b72bc
8a11330
cf1c282
bd03edc
96304ea
8de7d03
d25e6b5
ea6b55d
c4c4995
76f2f3b
32b783f
ea3050f
71b3c32
7a53f10
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -28,13 +28,15 @@ | |
| "location": "https://contoso.com/operationstatus", | ||
| "Azure-AsyncOperation": "https://contoso.com/operationstatus", | ||
| "Operation-Location": "https://contoso.com/operationstatus", | ||
| "retry-after": "30" | ||
| "Retry-After": "30" | ||
| }, | ||
| "body": { | ||
| "id": "/providers/Microsoft.Management/managementGroups/E7EC67B3-7657-4966-BFFC-41EFD36BAA09/providers/Microsoft.Quota/groupQuotas/groupquota1", | ||
| "type": "Microsoft.Quota/groupQuotas", | ||
| "name": "groupquota1", | ||
| "properties": { | ||
| "displayName": "GroupQuota1", | ||
| "groupType": "AllocationGroup", | ||
| "provisioningState": "Accepted" | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. displayName should appear in the response, since it was in the request #Resolved
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Added in the response |
||
| } | ||
tejasm-microsoft marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| } | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -30,13 +30,15 @@ | |
| "location": "https://contoso.com/operationstatus", | ||
| "Azure-AsyncOperation": "https://contoso.com/operationstatus", | ||
| "Operation-Location": "https://contoso.com/operationstatus", | ||
| "retry-after": "30" | ||
| "Retry-After": "30" | ||
| }, | ||
| "body": { | ||
| "id": "/providers/Microsoft.Management/managementGroups/E7EC67B3-7657-4966-BFFC-41EFD36BAA09/providers/Microsoft.Quota/groupQuotas/groupquota1/resourceProviders/Microsoft.Compute/locationSettings/eastus", | ||
| "type": "Microsoft.Quota/groupQuotas/locationSettings", | ||
| "name": "eastus", | ||
| "properties": { | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We'll need the enforcedGroupName here as well.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yes, all properties supplied in the initial PUT request, and any readonly properties, should appear in the 201 response if possible, as well as the 200 |
||
| "enforcementEnabled": "Enabled", | ||
| "enforcedGroupName": "groupquota1-eastus", | ||
| "provisioningState": "InProgress" | ||
| } | ||
| } | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wouldn't groupType also be returned in your 201 response? #Resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added