Merge branch 'release/latest'

Author: nmetulev

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@microsoft/mgt",
-  "version": "1.2.3",
+  "version": "1.2.4",
   "description": "The Microsoft Graph Toolkit",
   "keywords": [
     "microsoft graph",

src/components/mgt-get/mgt-get.ts

@@ -140,7 +140,7 @@ export class MgtGet extends MgtTemplatedComponent {
    * trigger the element to update.
    */
   protected render() {
-    if (this.isLoadingState) {
+    if (this.isLoadingState && !this.isPolling) {
       return this.renderTemplate('loading', null);
     } else if (this.error) {
       return this.renderTemplate('error', this.error);
@@ -179,8 +179,6 @@ export class MgtGet extends MgtTemplatedComponent {
       }
     } else if (this.response) {
       return this.renderTemplate('default', this.response) || html``;
-    } else if (this.isLoadingState) {
-      return this.renderTemplate('loading', null) || html``;
     } else {
       return html``;
     }
@@ -205,12 +203,14 @@ export class MgtGet extends MgtTemplatedComponent {
     if (this.resource) {
       try {
         let uri = this.resource;
-        let delta = false;
+        let isDeltaLink = false;
 
         // if we had a response earlier with a delta link, use it instead
         if (this.response && this.response['@odata.deltaLink']) {
           uri = this.response['@odata.deltaLink'];
-          delta = true;
+          isDeltaLink = true;
+        } else {
+          isDeltaLink = new URL(uri, 'https://graph.microsoft.com').pathname.endsWith('delta');
         }
 
         const graph = provider.graph.forComponent(this);
@@ -220,23 +220,23 @@ export class MgtGet extends MgtTemplatedComponent {
           request = request.middlewareOptions(prepScopes(...this.scopes));
         }
 
-        const response = await request.get();
+        let response = await request.get();
 
-        if (delta && this.response && Array.isArray(this.response.value) && Array.isArray(response.value)) {
+        if (isDeltaLink && this.response && Array.isArray(this.response.value) && Array.isArray(response.value)) {
           response.value = this.response.value.concat(response.value);
         }
 
-        if (!equals(this.response, response)) {
+        if (!this.isPolling && !equals(this.response, response)) {
           this.response = response;
         }
 
         // get more pages if there are available
-        if (this.response && Array.isArray(this.response.value) && this.response['@odata.nextLink']) {
+        if (response && Array.isArray(response.value) && response['@odata.nextLink']) {
           let pageCount = 1;
-          let page = this.response;
+          let page = response;
 
           while (
-            (pageCount < this.maxPages || this.maxPages <= 0 || this.pollingRate) &&
+            (pageCount < this.maxPages || this.maxPages <= 0 || (isDeltaLink && this.pollingRate)) &&
             page &&
             page['@odata.nextLink']
           ) {
@@ -247,11 +247,18 @@ export class MgtGet extends MgtTemplatedComponent {
               .version(this.version)
               .get();
             if (page && page.value && page.value.length) {
-              page.value = this.response.value.concat(page.value);
-              this.response = page;
+              page.value = response.value.concat(page.value);
+              response = page;
+              if (!this.isPolling) {
+                this.response = response;
+              }
             }
           }
         }
+
+        if (!equals(this.response, response)) {
+          this.response = response;
+        }
       } catch (e) {
         this.error = e;
       }

src/components/mgt-person/mgt-person.scss

@@ -74,18 +74,24 @@ mgt-person .user-avatar {
     border-radius: 50%;
     font-weight: 400;
 
-    & .initials-text {
+    & .initials-text,
+    & .ms-Icon {
       cursor: default;
       margin-top: -1px;
       font-size: calc(var(--avatar-size, 48px) * 0.4);
     }
+
+    & .ms-Icon {
+      display: inline;
+    }
   }
 
   &.small {
     width: $avatar-size-s;
     height: $avatar-size-s;
 
-    & .initials-text {
+    & .initials-text,
+    & .ms-Icon {
       font-size: calc(var(--avatar-size-s, 24px) * 0.4);
     }
   }

src/components/mgt-person/mgt-person.ts

@@ -296,6 +296,7 @@ export class MgtPerson extends MgtTemplatedComponent {
       this.personDetails && this.personCardInteraction === PersonCardInteraction.none
         ? this.personDetails.displayName
         : '';
+
     const isLarge = this.showEmail && this.showName;
     const imageClasses = {
       initials: !imageSrc,
@@ -305,19 +306,31 @@ export class MgtPerson extends MgtTemplatedComponent {
     };
 
     let imageHtml: TemplateResult;
+
     if (imageSrc) {
       // render the image
       imageHtml = html`
         <img alt=${title} src=${imageSrc} />
       `;
     } else if (this.personDetails) {
-      // render the initials
-      const initials = this.getInitials(this.personDetails);
+      // render the initials or person icon
+
       // add avatar background color
       imageClasses[this._personAvatarBg] = true;
+
+      const initials = this.getInitials(this.personDetails);
+      const initialsHtml =
+        initials && initials.length
+          ? html`
+              ${initials}
+            `
+          : html`
+              <i class="ms-Icon ms-Icon--Contact"></i>
+            `;
+
       imageHtml = html`
         <span class="initials-text" aria-label="${initials}">
-          ${initials}
+          ${initialsHtml}
         </span>
       `;
     } else {
@@ -518,7 +531,7 @@ export class MgtPerson extends MgtTemplatedComponent {
     if (!initials && person.displayName) {
       const name = person.displayName.split(/\s+/);
       for (let i = 0; i < 2 && i < name.length; i++) {
-        if (name[i][0] && name[i][0].match(/[a-z]/i)) {
+        if (name[i][0] && name[i][0].match(/\p{L}/gu)) {
           // check if letter
           initials += name[i][0].toUpperCase();
         }

src/utils/version.ts

@@ -8,4 +8,4 @@
 // THIS FILE IS AUTO GENERATED
 // ANY CHANGES WILL BE LOST DURING BUILD
 
-export const PACKAGE_VERSION = '1.2.3';
+export const PACKAGE_VERSION = '1.2.4';