[AutoPR @azure/arm-maps] [Hub Generated] Review request for Microsoft.Maps to add version stable/2021-02-01

sdk/maps/arm-maps/LICENSE.txt

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2018 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

sdk/maps/arm-maps/README.md

@@ -1,6 +1,6 @@
-## Azure MapsManagementClient SDK for JavaScript
+## Azure AzureMapsManagementClient SDK for JavaScript
 
-This package contains an isomorphic SDK for MapsManagementClient.
+This package contains an isomorphic SDK for AzureMapsManagementClient.
 
 ### Currently supported environments
 
@@ -9,31 +9,31 @@ This package contains an isomorphic SDK for MapsManagementClient.
 
 ### How to Install
 
-```
+```bash
 npm install @azure/arm-maps
 ```
 
 ### How to use
 
-#### nodejs - Authentication, client creation and get accounts as an example written in TypeScript.
+#### nodejs - client creation and get accounts as an example written in TypeScript.
 
 ##### Install @azure/ms-rest-nodeauth
 
-```
-npm install @azure/ms-rest-nodeauth
+- Please install minimum version of `"@azure/ms-rest-nodeauth": "^3.0.0"`.
+```bash
+npm install @azure/ms-rest-nodeauth@"^3.0.0"
 ```
 
 ##### Sample code
 
-```ts
-import * as msRest from "@azure/ms-rest-js";
-import * as msRestAzure from "@azure/ms-rest-azure-js";
-import * as msRestNodeAuth from "@azure/ms-rest-nodeauth";
-import { MapsManagementClient, MapsManagementModels, MapsManagementMappers } from "@azure/arm-maps";
+While the below sample uses the interactive login, other authentication options can be found in the [README.md file of @azure/ms-rest-nodeauth](https://www.npmjs.com/package/@azure/ms-rest-nodeauth) package
+```typescript
+const msRestNodeAuth = require("@azure/ms-rest-nodeauth");
+const { AzureMapsManagementClient } = require("@azure/arm-maps");
 const subscriptionId = process.env["AZURE_SUBSCRIPTION_ID"];
 
 msRestNodeAuth.interactiveLogin().then((creds) => {
-  const client = new MapsManagementClient(creds, subscriptionId);
+  const client = new AzureMapsManagementClient(creds, subscriptionId);
   const resourceGroupName = "testresourceGroupName";
   const accountName = "testaccountName";
   client.accounts.get(resourceGroupName, accountName).then((result) => {
@@ -49,7 +49,7 @@ msRestNodeAuth.interactiveLogin().then((creds) => {
 
 ##### Install @azure/ms-rest-browserauth
 
-```
+```bash
 npm install @azure/ms-rest-browserauth
 ```
 
@@ -78,7 +78,7 @@ See https://github.com/Azure/ms-rest-browserauth to learn how to authenticate to
           // may cause redirects
           authManager.login();
         }
-        const client = new Azure.ArmMaps.MapsManagementClient(res.creds, subscriptionId);
+        const client = new Azure.ArmMaps.AzureMapsManagementClient(res.creds, subscriptionId);
         const resourceGroupName = "testresourceGroupName";
         const accountName = "testaccountName";
         client.accounts.get(resourceGroupName, accountName).then((result) => {
@@ -99,5 +99,4 @@ See https://github.com/Azure/ms-rest-browserauth to learn how to authenticate to
 
 - [Microsoft Azure SDK for Javascript](https://github.com/Azure/azure-sdk-for-js)
 
-
-![Impressions](https://azure-sdk-impressions.azurewebsites.net/api/impressions/azure-sdk-for-js%2Fsdk%2Fmaps%2Farm-maps%2FREADME.png)
+![Impressions](https://azure-sdk-impressions.azurewebsites.net/api/impressions/azure-sdk-for-js/sdk/maps/arm-maps/README.png)

sdk/maps/arm-maps/package.json

@@ -1,12 +1,12 @@
 {
   "name": "@azure/arm-maps",
   "author": "Microsoft Corporation",
-  "description": "MapsManagementClient Library with typescript type definitions for node.js and browser.",
+  "description": "AzureMapsManagementClient Library with typescript type definitions for node.js and browser.",
   "version": "1.1.0",
   "dependencies": {
-    "@azure/ms-rest-azure-js": "^1.1.0",
-    "@azure/ms-rest-js": "^1.1.0",
-    "tslib": "^1.9.3"
+    "@azure/ms-rest-azure-js": "^2.0.1",
+    "@azure/ms-rest-js": "^2.0.4",
+    "tslib": "^1.10.0"
   },
   "keywords": [
     "node",
@@ -17,21 +17,22 @@
   ],
   "license": "MIT",
   "main": "./dist/arm-maps.js",
-  "module": "./esm/mapsManagementClient.js",
-  "types": "./esm/mapsManagementClient.d.ts",
+  "module": "./esm/azureMapsManagementClient.js",
+  "types": "./esm/azureMapsManagementClient.d.ts",
   "devDependencies": {
-    "typescript": "^3.1.1",
-    "rollup": "^0.66.2",
-    "rollup-plugin-node-resolve": "^3.4.0",
-    "uglify-js": "^3.4.9"
+    "typescript": "^3.5.3",
+    "rollup": "^1.18.0",
+    "rollup-plugin-node-resolve": "^5.2.0",
+    "rollup-plugin-sourcemaps": "^0.4.2",
+    "uglify-js": "^3.6.0"
   },
-  "homepage": "https://github.com/azure/azure-sdk-for-js/tree/master/sdk/maps/arm-maps",
+  "homepage": "https://github.com/Azure/azure-sdk-for-js/tree/master/sdk/maps/arm-maps",
   "repository": {
     "type": "git",
-    "url": "https://github.com/azure/azure-sdk-for-js.git"
+    "url": "https://github.com/Azure/azure-sdk-for-js.git"
   },
   "bugs": {
-    "url": "https://github.com/azure/azure-sdk-for-js/issues"
+    "url": "https://github.com/Azure/azure-sdk-for-js/issues"
   },
   "files": [
     "dist/**/*.js",
@@ -43,6 +44,7 @@
     "esm/**/*.d.ts",
     "esm/**/*.d.ts.map",
     "src/**/*.ts",
+    "README.md",
     "rollup.config.js",
     "tsconfig.json"
   ],

sdk/maps/arm-maps/rollup.config.js

@@ -1,10 +1,16 @@
+import rollup from "rollup";
 import nodeResolve from "rollup-plugin-node-resolve";
+import sourcemaps from "rollup-plugin-sourcemaps";
+
 /**
- * @type {import('rollup').RollupFileOptions}
+ * @type {rollup.RollupFileOptions}
  */
 const config = {
-  input: './esm/mapsManagementClient.js',
-  external: ["@azure/ms-rest-js", "@azure/ms-rest-azure-js"],
+  input: "./esm/azureMapsManagementClient.js",
+  external: [
+    "@azure/ms-rest-js",
+    "@azure/ms-rest-azure-js"
+  ],
   output: {
     file: "./dist/arm-maps.js",
     format: "umd",
@@ -15,17 +21,17 @@ const config = {
       "@azure/ms-rest-azure-js": "msRestAzure"
     },
     banner: `/*
- * Copyright (c) Microsoft Corporation. All rights reserved.
- * Licensed under the MIT License. See License.txt in the project root for
- * license information.
+ * 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.
+ * Changes may cause incorrect behavior and will be lost if the code is regenerated.
  */`
   },
   plugins: [
-    nodeResolve({ module: true })
+    nodeResolve({ mainFields: ['module', 'main'] }),
+    sourcemaps()
   ]
 };
+
 export default config;

sdk/maps/arm-maps/src/mapsManagementClient.ts → sdk/maps/arm-maps/src/azureMapsManagementClient.ts

@@ -1,7 +1,6 @@
 /*
- * Copyright (c) Microsoft Corporation. All rights reserved.
- * Licensed under the MIT License. See License.txt in the project root for
- * license information.
+ * 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
@@ -12,32 +11,35 @@ import * as msRest from "@azure/ms-rest-js";
 import * as Models from "./models";
 import * as Mappers from "./models/mappers";
 import * as operations from "./operations";
-import { MapsManagementClientContext } from "./mapsManagementClientContext";
+import { AzureMapsManagementClientContext } from "./azureMapsManagementClientContext";
 
 
-class MapsManagementClient extends MapsManagementClientContext {
+class AzureMapsManagementClient extends AzureMapsManagementClientContext {
   // Operation groups
   accounts: operations.Accounts;
+  maps: operations.Maps;
+  creators: operations.Creators;
 
   /**
-   * Initializes a new instance of the MapsManagementClient class.
+   * Initializes a new instance of the AzureMapsManagementClient class.
    * @param credentials Credentials needed for the client to connect to Azure.
-   * @param subscriptionId Subscription credentials which uniquely identify Microsoft Azure
-   * subscription. The subscription ID forms part of the URI for every service call.
+   * @param subscriptionId The ID of the target subscription.
    * @param [options] The parameter options
    */
-  constructor(credentials: msRest.ServiceClientCredentials, subscriptionId: string, options?: Models.MapsManagementClientOptions) {
+  constructor(credentials: msRest.ServiceClientCredentials, subscriptionId: string, options?: Models.AzureMapsManagementClientOptions) {
     super(credentials, subscriptionId, options);
     this.accounts = new operations.Accounts(this);
+    this.maps = new operations.Maps(this);
+    this.creators = new operations.Creators(this);
   }
 }
 
 // Operation Specifications
 
 export {
-  MapsManagementClient,
-  MapsManagementClientContext,
-  Models as MapsManagementModels,
-  Mappers as MapsManagementMappers
+  AzureMapsManagementClient,
+  AzureMapsManagementClientContext,
+  Models as AzureMapsManagementModels,
+  Mappers as AzureMapsManagementMappers
 };
 export * from "./operations";

sdk/maps/arm-maps/src/mapsManagementClientContext.ts → sdk/maps/arm-maps/src/azureMapsManagementClientContext.ts

@@ -1,7 +1,6 @@
 /*
- * Copyright (c) Microsoft Corporation. All rights reserved.
- * Licensed under the MIT License. See License.txt in the project root for
- * license information.
+ * 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
@@ -13,21 +12,20 @@ import * as msRest from "@azure/ms-rest-js";
 import * as msRestAzure from "@azure/ms-rest-azure-js";
 
 const packageName = "@azure/arm-maps";
-const packageVersion = "0.1.0";
+const packageVersion = "1.1.0";
 
-export class MapsManagementClientContext extends msRestAzure.AzureServiceClient {
+export class AzureMapsManagementClientContext extends msRestAzure.AzureServiceClient {
   credentials: msRest.ServiceClientCredentials;
   subscriptionId: string;
   apiVersion?: string;
 
   /**
-   * Initializes a new instance of the MapsManagementClient class.
+   * Initializes a new instance of the AzureMapsManagementClient class.
    * @param credentials Credentials needed for the client to connect to Azure.
-   * @param subscriptionId Subscription credentials which uniquely identify Microsoft Azure
-   * subscription. The subscription ID forms part of the URI for every service call.
+   * @param subscriptionId The ID of the target subscription.
    * @param [options] The parameter options
    */
-  constructor(credentials: msRest.ServiceClientCredentials, subscriptionId: string, options?: Models.MapsManagementClientOptions) {
+  constructor(credentials: msRest.ServiceClientCredentials, subscriptionId: string, options?: Models.AzureMapsManagementClientOptions) {
     if (credentials == undefined) {
       throw new Error('\'credentials\' cannot be null.');
     }
@@ -45,7 +43,7 @@ export class MapsManagementClientContext extends msRestAzure.AzureServiceClient
 
     super(credentials, options);
 
-    this.apiVersion = '2018-05-01';
+    this.apiVersion = '2021-02-01';
     this.acceptLanguage = 'en-US';
     this.longRunningOperationRetryTimeout = 30;
     this.baseUri = options.baseUri || this.baseUri || "https://management.azure.com";

sdk/maps/arm-maps/src/models/accountsMappers.ts

@@ -1,28 +1,28 @@
 /*
- * Copyright (c) Microsoft Corporation. All rights reserved.
- * Licensed under the MIT License. See License.txt in the project root for
- * license information.
+ * 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.
+ * Changes may cause incorrect behavior and will be lost if the code is regenerated.
  */
 
 export {
-  MapsAccountCreateParameters,
-  Sku,
-  MapsAccount,
-  Resource,
+  AzureEntityResource,
   BaseResource,
-  ErrorModel,
-  ErrorDetailsItem,
-  MapsAccountUpdateParameters,
-  MapsAccounts,
-  MapsAccountsMoveRequest,
+  Creator,
+  CreatorProperties,
+  ErrorAdditionalInfo,
+  ErrorDetail,
+  ErrorResponse,
+  MapsAccount,
   MapsAccountKeys,
+  MapsAccountProperties,
+  MapsAccounts,
+  MapsAccountUpdateParameters,
   MapsKeySpecification,
-  MapsOperations,
-  MapsOperationsValueItem,
-  MapsOperationsValueItemDisplay
+  ProxyResource,
+  Resource,
+  Sku,
+  SystemData,
+  TrackedResource
 } from "../models/mappers";
-

sdk/maps/arm-maps/src/models/creatorsMappers.ts

@@ -0,0 +1,26 @@
+/*
+ * 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 {
+  AzureEntityResource,
+  BaseResource,
+  Creator,
+  CreatorList,
+  CreatorProperties,
+  CreatorUpdateParameters,
+  ErrorAdditionalInfo,
+  ErrorDetail,
+  ErrorResponse,
+  MapsAccount,
+  MapsAccountProperties,
+  ProxyResource,
+  Resource,
+  Sku,
+  SystemData,
+  TrackedResource
+} from "../models/mappers";