Sync Network release branch with master #7764
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* update comment * update
#7377) * AdministratorName parameter should take constant value defined in enum * Code review fixes * Code review fixes * Fix tabs and spaces * Fix example with get * revert example with get * Fix examples
* addtdebyokapiswaggers * add inaccessible database and inaccessible managed database swaggers and examples * fix ci error * add all files * fix MA * 1 * fix some example error * update readme.md * modify readme.md * fix avacado * fix avacado 2 * addtdebyokapiswaggers * add inaccessible database and inaccessible managed database swaggers and examples * fix ci error * add all files * fix MA * 1 * fix some example error * update readme.md * modify readme.md * fix avacado * fix avacado 2 * fix prettier
* Update databaseVulnerabilityAssessments.json Updated storageContainerSasKey explaining that both read and write access is required in the SAS token. * Fixed documentation for storageAccountAccessKey parameter in SQL templates
* adding base examples and swagger files for undocumented api versions * update api version in example files to match swagger spec * update indentation to 2 instead of 4 * add new api versions to readme file * correct reference to 2014 api version * ran prettier on all example files
…d adding 'Spot' priority (#7748)
* First commit for new version folder with old files Please diff later commits with this one for the changes. * Update version. Add Jit requests. Try fix Model validation. * Update operation Id * Operations read, refresh permission, PR comments. * Application definition artifact. * Fix build * Fix build * Jit Patch * Post async 202 * Fix example model valiation. * Fix PR comments. Remove update access since it should be internal only(used by PIM team) * Capitalization to fix build. * Read only properties * Removed application definition artifact, which is internal and don't need a swagger.
* Copy 2018-07-01-preview to 2019-07-01 * Update spec and examples * Change rg name in examples * Fix readme path
anton-evseev
requested review from
DavidLiCIG,
hyonholee,
jaredmoo,
matthchr,
MikhailTryakhov and
schaabs
as code owners
anton-evseev
requested review from
Tiano2017,
xingwu1 and
yangyuan
as code owners
Automation for azure-sdk-for-pythonThis PR contains more than 3 context, SDK generation is not enabled. Contexts found:
|
Automation for azure-sdk-for-goThis PR contains more than 3 context, SDK generation is not enabled. Contexts found:
|
5 tasks
…gers (#7734) * Remove from 'track2' copy the swagger for 2019-05-06-Preview * Remove from 'track2' copy the swagger for 2019-05-06-Preview for the service swagger
* Remove references to CosmosDB and make data source type a generated enum * Remove named entity skill and associated models * Expand the list of EntityRecognitionSkillLanguage * Trial using opt-in-extensible-enums * Use extensible enum for language * Use extensible enum for TokenizerName * Use extensible enum for TokenFilterName * Use extensible enum for CharFilterName * Use extensible enum for RegexFlags * Fix missing bool setting * Remove RegexFlags * Add description for language properties * Add URL to description * Apply x-ms-enum fixes to 2019-05-06 'track2' swagger * Fix model validation spelling errors * Actually fix model validation spelling errors via custom-words.txt * Opt-in extensible enums globally * Model failing validation enums as string (still customized in SDK side) * Remove the example in swagger specs that cause validation error for data sources * Remove NamedEntityRecognitionSkill from the yet to ship 'track2' SDK version of swagger * Fix swapping of name and description
|
Can one of the admins verify this patch? |
|
ModelValidation errors are from other RP specs |
* [QnA Maker] Update API data fields. * camel casing
* Added enableCustomerAnalytics Flag to create SapMonitor * Added the new flag to the correct file
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Need to get fix for CI into Network branch (for #7747 )