Skip to content

Commit

Permalink
CodeGen from PR 14035 in Azure/azure-rest-api-specs
Browse files Browse the repository at this point in the history
Fix to support nextLink with post calls for python sdk (Azure#14035)

* single example with post nextLinl

* address suggestions

* remove examples

* Add nextLink param

* Post on nextLink for all requests

* Add api-version as param

* update readme

* prettier fix

* fix lint err try

* rename operation id

* fix generation for python

* fix casing of next link

* Remove package

* change one next link op to normal path

* readme.python config

* Update readme.python.md

Co-authored-by: Namrata Jagasia <[email protected]>
Co-authored-by: iscai-msft <[email protected]>
Co-authored-by: Zed <[email protected]>
Co-authored-by: msyyc <[email protected]>
  • Loading branch information
5 people committed Jul 16, 2021
1 parent 88a3220 commit 1ae5efd
Show file tree
Hide file tree
Showing 18 changed files with 2,461 additions and 954 deletions.
2 changes: 1 addition & 1 deletion sdk/policyinsights/arm-policyinsights/LICENSE.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
The MIT License (MIT)

Copyright (c) 2020 Microsoft
Copyright (c) 2021 Microsoft

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
Expand Down
8 changes: 3 additions & 5 deletions sdk/policyinsights/arm-policyinsights/README.md
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
## Azure PolicyInsightsClient SDK for JavaScript

This package contains an isomorphic SDK (runs both in Node.js and in browsers) for PolicyInsightsClient.
This package contains an isomorphic SDK (runs both in node.js and in browsers) for PolicyInsightsClient.

### Currently supported environments

- [LTS versions of Node.js](https://nodejs.org/about/releases/)
- Latest versions of Safari, Chrome, Edge, and Firefox.
- Latest versions of Safari, Chrome, Edge and Firefox.

### Prerequisites

Expand All @@ -21,7 +21,6 @@ Install both packages using the below command:
```bash
npm install --save @azure/arm-policyinsights @azure/identity
```

> **Note**: You may have used either `@azure/ms-rest-nodeauth` or `@azure/ms-rest-browserauth` in the past. These packages are in maintenance mode receiving critical bug fixes, but no new features.
If you are on a [Node.js that has LTS status](https://nodejs.org/about/releases/), or are writing a client side browser application, we strongly encourage you to upgrade to `@azure/identity` which uses the latest versions of Azure Active Directory and MSAL APIs and provides more authentication options.

Expand All @@ -37,7 +36,6 @@ If you are on a [Node.js that has LTS status](https://nodejs.org/about/releases/

In the below samples, we pass the credential and the Azure subscription id to instantiate the client.
Once the client is created, explore the operations on it either in your favorite editor or in our [API reference documentation](https://docs.microsoft.com/javascript/api) to get started.

#### nodejs - Authentication, client creation, and listForManagementGroup remediations as an example written in JavaScript.

##### Sample code
Expand Down Expand Up @@ -88,7 +86,7 @@ In browser applications, we recommend using the `InteractiveBrowserCredential` t
const credential = new InteractiveBrowserCredential(
{
clientId: "<client id for your Azure AD app>",
tenantId: "<optional tenant for your organization>"
tenant: "<optional tenant for your organization>"
});
const client = new Azure.ArmPolicyinsights.PolicyInsightsClient(creds, subscriptionId);
const managementGroupId = "testmanagementGroupId";
Expand Down
2 changes: 1 addition & 1 deletion sdk/policyinsights/arm-policyinsights/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
"rollup-plugin-sourcemaps": "^0.4.2",
"uglify-js": "^3.6.0"
},
"homepage": "https://github.com/Azure/azure-sdk-for-js/tree/main/sdk/policyinsights/arm-policyinsights",
"homepage": "https://github.com/Azure/azure-sdk-for-js/tree/master/sdk/policyinsights/arm-policyinsights",
"repository": {
"type": "git",
"url": "https://github.com/Azure/azure-sdk-for-js.git"
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
/*
* Copyright (c) Microsoft Corporation.
* Licensed under the MIT License.
*
* Code generated by Microsoft (R) AutoRest Code Generator.
* Changes may cause incorrect behavior and will be lost if the code is regenerated.
*/

export {
Attestation,
AttestationEvidence,
AttestationListResult,
AzureEntityResource,
BaseResource,
ErrorDefinition,
ErrorResponse,
ProxyResource,
Remediation,
RemediationDeploymentSummary,
RemediationFilters,
Resource,
SystemData,
TrackedResource,
TypedErrorInfo
} from "../models/mappers";
Loading

0 comments on commit 1ae5efd

Please sign in to comment.