Azure / Azure/azure-rest-api-specs

[appconfiguration] "RegenerateKeyParameters.id" should be required

Open
#10,188 1 comment 0 reactions 0 assignees View on GitHub
App Configuration question Service Attention
Dominant language
TypeSpec
Stars
3.1k
Forks
5.9k
Avg merge
2d 22h
Merged PRs (30d)
444

Description

This is Yeming from Azure PowerShell team. While we are working on the generated module Az.AppConfiguration, we found what looks like a bug in swagger that `RegenerateKeyParameters.id` is [not marked as required](https://github.com/Azure/azure-rest-api-specs/blob/master/specification/appconfiguration/resource-manager/Microsoft.AppConfiguration/preview/2019-11-01-preview/appconfiguration.json#L1151) as it should be.

@jimmyca15 I saw you updated the spec lastly. Could you help correct that? Thanks

(We have a workaround so this is not a blocker)

Contributor guide

Open the contributing guide

Research direction

Open specification/appconfiguration/resource-manager/Microsoft.AppConfiguration/preview/2019-11-01-preview/appconfiguration.json around line 1151, where RegenerateKeyParameters.id is defined. Confirm how nearby required properties are declared and update the specification so id is required; done means the repository’s specification validation passes.

Written by the indexing model from the issue text.

Assessment

Tech stack
openapi
Domain
api
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.