@microsoft/mgt-element - npm Package Compare versions

Comparing version 2.2.0-next.file-picker.bcc3b0d to 2.2.0-next.msal2provider.178cdd7

dist/es6/components/baseComponent.js

@@ -224,8 +224,11 @@ /**
             const provider = Providers.globalProvider;
-            if (provider.state == ProviderState.SignedOut) {
+            if (!provider) {
+                return Promise.resolve();
+            }
+            if (provider.state === ProviderState.SignedOut) {
                 // Signed out, clear the component state
                 this.clearState();
-                return Promise.resolve();
+                return;
             }
-            else if (provider.state == ProviderState.Loading) {
+            else if (provider.state === ProviderState.Loading) {
                 // The provider state is indeterminate. Do nothing.
@@ -232,0 +235,0 @@ return Promise.resolve();

dist/es6/components/templatedComponent.d.ts

@@ -17,2 +17,4 @@ /**
  * @extends {MgtBaseComponent}
+ *
+ * @fires templateRendered - fires when a template is rendered
  */
@@ -19,0 +21,0 @@ export declare abstract class MgtTemplatedComponent extends MgtBaseComponent {

dist/es6/components/templatedComponent.js

@@ -18,2 +18,4 @@ /**
  * @extends {MgtBaseComponent}
+ *
+ * @fires templateRendered - fires when a template is rendered
  */
@@ -20,0 +22,0 @@ export class MgtTemplatedComponent extends MgtBaseComponent {

dist/es6/Graph.js

@@ -124,3 +124,3 @@ /**
         new TelemetryHandler(),
-        new SdkVersionMiddleware(PACKAGE_VERSION),
+        new SdkVersionMiddleware(PACKAGE_VERSION, provider.name),
         new HTTPMessageHandler()
@@ -127,0 +127,0 @@ ];

dist/es6/mock/MockProvider.d.ts

@@ -46,3 +46,10 @@ /**
     getAccessToken(): Promise<string>;
+    /**
+     * Name used for analytics
+     *
+     * @readonly
+     * @memberof IProvider
+     */
+    get name(): string;
 }
 //# sourceMappingURL=MockProvider.d.ts.map

dist/es6/mock/MockProvider.js

@@ -76,3 +76,12 @@ /**
     }
+    /**
+     * Name used for analytics
+     *
+     * @readonly
+     * @memberof IProvider
+     */
+    get name() {
+        return 'MgtMockProvider';
+    }
 }
 //# sourceMappingURL=MockProvider.js.map

dist/es6/providers/IProvider.d.ts

@@ -37,2 +37,9 @@ /**
     get state(): ProviderState;
+    /**
+     * Name used for analytics
+     *
+     * @readonly
+     * @memberof IProvider
+     */
+    get name(): string;
     constructor();
@@ -75,3 +82,3 @@ /**
     /**
-     * uses scopes to recieve access token
+     * uses scopes to receive access token
      *
@@ -78,0 +85,0 @@ * @param {...string[]} scopes

dist/es6/providers/IProvider.js

@@ -32,2 +32,11 @@ /**
     /**
+     * Name used for analytics
+     *
+     * @readonly
+     * @memberof IProvider
+     */
+    get name() {
+        return 'MgtIProvider';
+    }
+    /**
      * sets state of Provider and fires loginchangedDispatcher
@@ -63,3 +72,3 @@ *
     /**
-     * uses scopes to recieve access token
+     * uses scopes to receive access token
      *
@@ -66,0 +75,0 @@ * @param {...string[]} scopes

dist/es6/providers/SimpleProvider.d.ts

@@ -20,2 +20,9 @@ /**
     private _logoutHandler;
+    /**
+     * Name used for analytics
+     *
+     * @readonly
+     * @memberof IProvider
+     */
+    get name(): string;
     constructor(getAccessTokenHandler: (scopes: string[]) => Promise<string>, loginHandler?: () => Promise<void>, logoutHandler?: () => Promise<void>);
@@ -22,0 +29,0 @@ /**

dist/es6/providers/SimpleProvider.js

@@ -25,2 +25,11 @@ /**
     /**
+     * Name used for analytics
+     *
+     * @readonly
+     * @memberof IProvider
+     */
+    get name() {
+        return 'MgtSimpleProvider';
+    }
+    /**
      * Invokes the getAccessToken function
@@ -27,0 +36,0 @@ *

dist/es6/utils/Batch.js

@@ -130,2 +130,5 @@ /**
                 }
+                else if (r.headers['Content-Type'].includes('image/pjpeg')) {
+                    response.content = 'data:image/pjpeg;base64,' + r.body;
+                }
                 else {
@@ -132,0 +135,0 @@ response.content = r.body;

dist/es6/utils/SdkVersionMiddleware.d.ts

@@ -22,3 +22,4 @@ /**
     private _packageVersion;
-    constructor(packageVersion: string);
+    private _providerName;
+    constructor(packageVersion: string, providerName?: string);
     execute(context: Context): Promise<void>;
@@ -25,0 +26,0 @@ /**

dist/es6/utils/SdkVersionMiddleware.js

@@ -26,4 +26,5 @@ /**
 export class SdkVersionMiddleware {
-    constructor(packageVersion) {
+    constructor(packageVersion, providerName) {
         this._packageVersion = packageVersion;
+        this._providerName = providerName;
     }
@@ -41,2 +42,6 @@ // tslint:disable-next-line: completed-docs
                 }
+                if (this._providerName) {
+                    const providerVersion = `${this._providerName}/${this._packageVersion}`;
+                    headerParts.push(providerVersion);
+                }
                 // Package version
@@ -43,0 +48,0 @@ const packageVersion = `mgt/${this._packageVersion}`;

package.json

 {
   "name": "@microsoft/mgt-element",
-  "version": "2.2.0-next.file-picker.bcc3b0d",
+  "version": "2.2.0-next.msal2provider.178cdd7",
   "description": "Microsoft Graph Toolkit base classes",
@@ -5,0 +5,0 @@ "homepage": "https://github.com/microsoftgraph/microsoft-graph-toolkit",

src/components/baseComponent.ts

@@ -231,7 +231,11 @@ /**
 
-    if (provider.state == ProviderState.SignedOut) {
+    if (!provider) {
+      return Promise.resolve();
+    }
+
+    if (provider.state === ProviderState.SignedOut) {
       // Signed out, clear the component state
       this.clearState();
-      return Promise.resolve();
-    } else if (provider.state == ProviderState.Loading) {
+      return;
+    } else if (provider.state === ProviderState.Loading) {
       // The provider state is indeterminate. Do nothing.
@@ -238,0 +242,0 @@ return Promise.resolve();

src/components/templatedComponent.ts

@@ -38,2 +38,4 @@ /**
  * @extends {MgtBaseComponent}
+ *
+ * @fires templateRendered - fires when a template is rendered
  */
@@ -40,0 +42,0 @@ export abstract class MgtTemplatedComponent extends MgtBaseComponent {

src/Graph.ts

@@ -154,3 +154,3 @@ /**
     new TelemetryHandler(),
-    new SdkVersionMiddleware(PACKAGE_VERSION),
+    new SdkVersionMiddleware(PACKAGE_VERSION, provider.name),
     new HTTPMessageHandler()
@@ -157,0 +157,0 @@ ];

src/mock/MockProvider.ts

@@ -48,2 +48,3 @@ /**
   }
+
   /**
@@ -60,2 +61,3 @@ * sets Provider state to signed out
   }
+
   /**
@@ -70,2 +72,12 @@ * Promise returning token from graph.microsoft.com
   }
+
+  /**
+   * Name used for analytics
+   *
+   * @readonly
+   * @memberof IProvider
+   */
+  public get name() {
+    return 'MgtMockProvider';
+  }
 }

src/providers/IProvider.ts

@@ -42,2 +42,12 @@ /**
 
+  /**
+   * Name used for analytics
+   *
+   * @readonly
+   * @memberof IProvider
+   */
+  public get name() {
+    return 'MgtIProvider';
+  }
+
   constructor() {
@@ -96,3 +106,3 @@ this._state = ProviderState.Loading;
   /**
-   * uses scopes to recieve access token
+   * uses scopes to receive access token
    *
@@ -99,0 +109,0 @@ * @param {...string[]} scopes

src/providers/SimpleProvider.ts

@@ -24,2 +24,12 @@ /**
 
+  /**
+   * Name used for analytics
+   *
+   * @readonly
+   * @memberof IProvider
+   */
+  public get name() {
+    return 'MgtSimpleProvider';
+  }
+
   constructor(
@@ -26,0 +36,0 @@ getAccessTokenHandler: (scopes: string[]) => Promise<string>,

src/utils/Batch.ts

@@ -190,2 +190,4 @@ /**
         response.content = 'data:image/jpeg;base64,' + r.body;
+      } else if (r.headers['Content-Type'].includes('image/pjpeg')) {
+        response.content = 'data:image/pjpeg;base64,' + r.body;
       } else {
@@ -192,0 +194,0 @@ response.content = r.body;

src/utils/SdkVersionMiddleware.ts

@@ -26,5 +26,7 @@ /**
   private _packageVersion: string;
+  private _providerName: string;
 
-  constructor(packageVersion: string) {
+  constructor(packageVersion: string, providerName?: string) {
     this._packageVersion = packageVersion;
+    this._providerName = providerName;
   }
@@ -47,2 +49,7 @@
 
+      if (this._providerName) {
+        const providerVersion: string = `${this._providerName}/${this._packageVersion}`;
+        headerParts.push(providerVersion);
+      }
+
       // Package version
@@ -49,0 +56,0 @@ const packageVersion: string = `mgt/${this._packageVersion}`;

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

1010858

increased by28.7%

Lines of code

6998

increased by1.51%

No dependency changes