Skip to content

Conversation

@zikalino
Copy link

This is to fix autogeneration problem when subscription_id is duplicated and appears as subscription_id1 for the second time.

@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Jul 31, 2019

SDK Automation [Logs] (Generated from 10bf6ed, Iteration 3)

Succeeded Python: Azure/azure-sdk-for-python [Logs] [Diff]
Warning Go: test-repo-billy/azure-sdk-for-go [Logs] [Diff]

@AutorestCI
Copy link

AutorestCI commented Jul 31, 2019

Automation for azure-sdk-for-python

A PR has been created for you:
Azure/azure-sdk-for-python#6574

@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@AutorestCI
Copy link

AutorestCI commented Jul 31, 2019

Automation for azure-sdk-for-go

A PR has been created for you:
Azure/azure-sdk-for-go#5384

@zikalino zikalino requested a review from mahakjain314 July 31, 2019 04:24
"required": true,
"type": "string",
"description": "The ID of the target subscription.",
"minLength": 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do we require 'minLength' property?

Copy link
Author

@zikalino zikalino Jul 31, 2019

Choose a reason for hiding this comment

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

Just following common file here:

"SubscriptionIdParameter": {
"name": "subscriptionId",
"in": "path",
"required": true,
"type": "string",
"description": "The ID of the target subscription.",
"minLength": 1
},

It just indicates that it shouldn't be empty string.

Copy link
Contributor

Choose a reason for hiding this comment

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

Makes sense.

"type": "string",
"description": "The Azure subscription id."
"description": "The ID of the target subscription.",
"minLength": 1
Copy link
Member

Choose a reason for hiding this comment

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

since this changed stable version, does it require a new version?

@yungezz yungezz merged commit d80bab6 into master Jul 31, 2019
@yungezz yungezz deleted the fixing-alerts-management-subscription-id-issue branch July 31, 2019 13:02
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.

6 participants