Merge pull request #1142 from microsoftgraph/release/3.0.5

Release/3.0.5

Author: nikithauc

.github/workflows/ci_validation.yml

@@ -19,6 +19,23 @@ jobs:
       uses: actions/setup-node@v3
       with:
         node-version: ${{ matrix.node-version }}
-    - run: npm ci
-    - run: npm run build --if-present
-    - run: npm test
+    - name: Install dependencies
+      run: npm ci
+      
+    - name: Build for node 14 and 16
+      if: matrix.node-version != '12.x'
+      run: npm run build --if-present
+      
+    - name: Build for node 12
+      if: matrix.node-version == '12.x'
+      run: npm run build:cjs && npm run build:es
+    
+    - name: Run unit tests
+      run: npm test
+
+    - name: Verify ESM compatibility
+      if: matrix.node-version == '16.x'
+      working-directory: './test-esm'
+      run: |
+        npm ci
+        npm test

.github/workflows/projectbot.yml

@@ -11,7 +11,7 @@ jobs:
     steps:
       - name: Generate token
         id: generate_token
-        uses: tibdex/github-app-token@f717b5ecd4534d3c4df4ce9b5c1c2214f0f7cd06
+        uses: tibdex/github-app-token@021a2405c7f990db57f5eae5397423dcc554159c
         with:
           app_id: ${{ secrets.GRAPHBOT_APP_ID }}
           private_key: ${{ secrets.GRAPHBOT_APP_PEM }}

.gitignore

@@ -31,3 +31,5 @@ test/development/secrets.ts
 .idea/*
 
 testResult.xml
+
+test-esm/lib/*

.npmignore

@@ -2,4 +2,5 @@ src/
 scripts/
 design/
 changelogs/
-testResult.xml
+testResult.xml
+test-esm/

README.md

@@ -205,6 +205,11 @@ Please see the [contributing guidelines](CONTRIBUTING.md).
 -   [Microsoft Graph Toolkit: UI Components and Authentication Providers for Microsoft Graph](https://docs.microsoft.com/graph/toolkit/overview)
 -   [Office Dev Center](http://dev.office.com/)
 
+## Tips and Tricks
+
+- [Microsoft Graph SDK `n.call is not a function` by Lee Ford](https://www.lee-ford.co.uk/posts/graph-sdk-is-not-a-function/)
+- [Example of using the Graph JS library with ESM and `importmaps` ](https://github.com/waldekmastykarz/js-graph-101/blob/main/index_esm.html)
+
 ## Third Party Notices
 
 See [Third Party Notices](./THIRD%20PARTY%20NOTICES) for information on the packages that are included in the [package.json](./package.json)

docs/AuthCodeMSALBrowserAuthenticationProvider.md

@@ -34,19 +34,19 @@ Using npm:
     import { AuthCodeMSALBrowserAuthenticationProvider, AuthCodeMSALBrowserAuthenticationProviderOptions } from "@microsoft/microsoft-graph-client/authProviders/authCodeMsalBrowser";
     import { Client } from "@microsoft/microsoft-graph-client";
 
-    const options:AuthCodeMSALBrowserAuthenticationProviderOptions: {
+    const options: AuthCodeMSALBrowserAuthenticationProviderOptions = {
         account: account, // the AccountInfo instance to acquire the token for.
-        interactionType: InteractionType.PopUp , // msal-browser InteractionType
+        interactionType: InteractionType.Popup, // msal-browser InteractionType
         scopes: ["user.read", "mail.send"] // example of the scopes to be passed
-    }
+    };
 
     // Pass the PublicClientApplication instance from step 2 to create AuthCodeMSALBrowserAuthenticationProvider instance
-    const authProvider: new AuthCodeMSALBrowserAuthenticationProvider(publicClientApplication, options),
+    const authProvider = new AuthCodeMSALBrowserAuthenticationProvider(publicClientApplication, options);
 
 
     // Initialize the Graph client
     const graphClient = Client.initWithMiddleware({
-        authprovider
+        authProvider
     });
 
 ```