Skip to content

Navigation Menu

Explore
By company size
By use case
By industry
View all solutions
Topics
- AI
- DevOps
- Security
- Software Development
- View all
Explore
- GitHub Sponsors
  Fund open source developers
- The ReadME Project
  GitHub community articles
Repositories
- Enterprise platform
  AI-powered developer platform
Available add-ons
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

Azure / azure-rest-api-specs Public

Notifications You must be signed in to change notification settings
Fork 5.1k
Star 2.7k

Code
Issues 1.7k
Pull requests 780
Discussions
Actions
Projects
Wiki
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Discussions
Actions
Projects
Wiki
Security
Insights

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.

Sign up for GitHub

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

Jump to bottom

[Do Not Merge] Add API for adding external storage account configurations to a web app #3423

Closed

ebencarek wants to merge 24 commits into Azure:master from ebencarek:master

Closed

[Do Not Merge] Add API for adding external storage account configurations to a web app #3423

ebencarek wants to merge 24 commits into Azure:master from ebencarek:master

Conversation 6 Commits 24 Checks 0 Files changed

Conversation

Copy link

Contributor

ebencarek commented Jul 16, 2018 •

edited

Loading

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.

PR information

The title of the PR is clear and informative.
There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
If applicable, the PR references the bug/issue that it fixes.
Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

I have read the contribution guidelines.
My spec meets the review criteria:
- The spec conforms to the Swagger 2.0 specification.
- The spec follows the guidelines described in the Swagger checklist document.
- Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Sorry, something went wrong.

All reactions


          Add API for adding external storage account configurations to a web app

6485ba3

azuresdkci assigned jianghaolu

azuresdkci requested a review from jianghaolu

July 16, 2018 22:05

Copy link

AutorestCI commented Jul 16, 2018 •

edited

Loading

Automation for azure-sdk-for-python

This PR contains more than 3 context, SDK generation is not enabled. Contexts found:

recoveryservicesbackup/resource-manager
automation/resource-manager
streamanalytics/resource-manager
storage/resource-manager
apimanagement/resource-manager
azsadmin/resource-manager/network/Microsoft.Network.Admin
compute/resource-manager
hanaonazure/resource-manager
alertsmanagement/resource-manager
security/resource-manager
iothub/resource-manager
hdinsight/resource-manager
cognitiveservices/resource-manager
powerbiembedded/resource-manager
web/resource-manager
containerservices/resource-manager
search/data-plane/Microsoft.Azure.Search.Service
botservice/resource-manager
cognitiveservices/data-plane/LUIS/Authoring
servicefabricmesh/resource-manager
azsadmin/resource-manager/azurebridge/Microsoft.AzureBridge.Admin
sql/resource-manager

All reactions

Sorry, something went wrong.

Copy link

AutorestCI commented Jul 16, 2018 •

edited

Loading

Automation for azure-sdk-for-ruby

This PR contains more than 3 context, SDK generation is not enabled. Contexts found:

streamanalytics/resource-manager
cognitiveservices/resource-manager
storage/resource-manager
web/resource-manager
botservice/resource-manager
powerbiembedded/resource-manager
alertsmanagement/resource-manager
servicefabricmesh/resource-manager
sql/resource-manager
recoveryservicesbackup/resource-manager
cognitiveservices/data-plane/LUIS/Authoring
apimanagement/resource-manager
azsadmin/resource-manager/network/Microsoft.Network.Admin
azsadmin/resource-manager/azurebridge/Microsoft.AzureBridge.Admin
automation/resource-manager
hdinsight/resource-manager
search/data-plane/Microsoft.Azure.Search.Service
iothub/resource-manager
compute/resource-manager
hanaonazure/resource-manager
containerservices/resource-manager
security/resource-manager

All reactions

Sorry, something went wrong.

AutorestCI mentioned this pull request

[AutoPR web/resource-manager] [Do Not Merge] Add API for adding external storage account configurations to a web app Azure/azure-sdk-for-go#2237

Closed

Copy link

AutorestCI commented Jul 16, 2018 •

edited

Loading

Automation for azure-sdk-for-go

This PR contains more than 3 context, SDK generation is not enabled. Contexts found:

azsadmin/resource-manager/network/Microsoft.Network.Admin
apimanagement/resource-manager
search/data-plane/Microsoft.Azure.Search.Service
security/resource-manager
compute/resource-manager
iothub/resource-manager
servicefabricmesh/resource-manager
sql/resource-manager
alertsmanagement/resource-manager
botservice/resource-manager
hdinsight/resource-manager
cognitiveservices/data-plane/LUIS/Authoring
web/resource-manager
powerbiembedded/resource-manager
azsadmin/resource-manager/azurebridge/Microsoft.AzureBridge.Admin
automation/resource-manager
containerservices/resource-manager
hanaonazure/resource-manager
storage/resource-manager
streamanalytics/resource-manager
cognitiveservices/resource-manager
recoveryservicesbackup/resource-manager

All reactions

Sorry, something went wrong.

Copy link

AutorestCI commented Jul 16, 2018 •

edited

Loading

Automation for azure-sdk-for-node

This PR contains more than 3 context, SDK generation is not enabled. Contexts found:

powerbiembedded/resource-manager
search/data-plane/Microsoft.Azure.Search.Service
azsadmin/resource-manager/network/Microsoft.Network.Admin
apimanagement/resource-manager
botservice/resource-manager
servicefabricmesh/resource-manager
cognitiveservices/data-plane/LUIS/Authoring
web/resource-manager
hanaonazure/resource-manager
automation/resource-manager
storage/resource-manager
sql/resource-manager
cognitiveservices/resource-manager
hdinsight/resource-manager
security/resource-manager
compute/resource-manager
azsadmin/resource-manager/azurebridge/Microsoft.AzureBridge.Admin
containerservices/resource-manager
streamanalytics/resource-manager
recoveryservicesbackup/resource-manager
iothub/resource-manager
alertsmanagement/resource-manager

All reactions

Sorry, something went wrong.

AutorestCI mentioned this pull request

[AutoPR web/resource-manager] [Do Not Merge] Add API for adding external storage account configurations to a web app Azure/azure-sdk-for-java#2228

Closed

Copy link

AutorestCI commented Jul 16, 2018 •

edited

Loading

Automation for azure-sdk-for-java

This PR contains more than 3 context, SDK generation is not enabled. Contexts found:

storage/resource-manager
azsadmin/resource-manager/azurebridge/Microsoft.AzureBridge.Admin
iothub/resource-manager
cognitiveservices/resource-manager
recoveryservicesbackup/resource-manager
streamanalytics/resource-manager
sql/resource-manager
hdinsight/resource-manager
apimanagement/resource-manager
alertsmanagement/resource-manager
compute/resource-manager
botservice/resource-manager
automation/resource-manager
powerbiembedded/resource-manager
servicefabricmesh/resource-manager
web/resource-manager
hanaonazure/resource-manager
azsadmin/resource-manager/network/Microsoft.Network.Admin
security/resource-manager
cognitiveservices/data-plane/LUIS/Authoring
containerservices/resource-manager
search/data-plane/Microsoft.Azure.Search.Service

All reactions

Sorry, something went wrong.

sergey-shandar and others added 4 commits

July 16, 2018 20:12


          Fix OAV error log (#3421)

d068d86

* Fix OAV error log

* remove test/model.js


          Bump hdinsight management Node.js package version to 0.4.0 (#3427)

e61491c


          [AKS] suppress two autorest ARM linter errors (#3388)

19450c4


          Update ReadMe.md by linking the announcements section (#3428)

b36d263

* Update ReadMe.md by linking the announcements section

* Update README.md

* Update README.md

jianghaolu approved these changes

View reviewed changes

solankisamir and others added 15 commits

July 17, 2018 14:25


          Add revision and Version Description to Contract (#3430)

e2b4303

* add revision description

* fixed missing file reference

* add the missing file


          Fixed operation ID for configuration operation (#3424)

7499d0e


          Changing schema from string to file (#3420)

40e3e99


           Added BackupValidateOperation API (#3395)

ccefbaf

* Added BackupValidateOperation API

* Added BackupValidateOperation API

* Fix casing in validateOperationResponse

* Fix casing in validateOperationResponse

* Fix casing in swagger example file path

* Use ValidateRequest instead of Resource<ValidateRequest>

* Update operationid of validate_Operation to Operation_Validate per feedback

* Update operationid of validate_Operation to Operation_Validate per feedback

* remove additional parameter api-version


          Security Center: Fixing documentation of auto-provision settings (#3429)

3316d44


          BugFix: defaultSinkRefs is an array of string (#3432)

fa000df


          Bump hdinsight Node.js package version to 0.5.0 (#3434)

fe957d8


          Bump recoveryservicesbackup Node.js package version to 3.0.0 (#3435)

50c1440


          Bump azure-arm-iothub Node.js package version to 2.0.1 (#3436)

c14b067


          AlertsManagement RP Public preview Swagger Spec (#3431)

31bbba8


          Modify Cognitive Search Swagger (#3386)

c4c8e6d

* Address CR comments

* Address CR comment

* Address PR comments in C#

* Fix syntax error

* Address CR comments


          Revert AccessUri description to make it correspond to server response (…

96d6b7f

…#3433)


           Update default package version for WebSiteManagementClient (#3415)

59180bb

* Updated webapps swagger with restore deleted site APIs

* Update default package version for WebSiteManagementClient


          Replace bad quotes (#3439)

07f0a74


          Adding swagger of sensitivity labels APIs. (#3373)

5dd98e6

* Swagger of sensitivity labels APIs

Adding swagger containing APIs of sensitivity labels, as long as usage examples of these APIs.

* Reference common definitions.

Reference common definitions.

deathly809 and others added 4 commits

July 18, 2018 14:22


          [Azure Stack] Add operations endpoint for AzureBridge and Network (#3438

de1a433


          [AKS] added PATCH endpoint for updating tags (#3443)


          Add API for adding external storage account configurations to a web app

a21611e


          Merge branch 'master' of https://github.com/ebencarek/azure-rest-api-…

6317a65

…specs

Need the updated default package version so Python SDK will be
generated.

ebencarek closed this

ebencarek mentioned this pull request

[Do Not Merge] Add API for adding external storage account configurations to a web app #3444

Merged

10 tasks

Copy link

Contributor Author

ebencarek commented Jul 19, 2018

Messed up commit history. Abandoned this PR, created new PR #3444.

All reactions

Sorry, something went wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

jianghaolu jianghaolu approved these changes

Assignees

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

16 participants

Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.

Footer

© 2024 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Docs
Contact

You can’t perform that action at this time.