Merge pull request #85 from pwa-builder/ignore_shortcut_entries

Ignore shortcut entries

Author: jgw96

lib/validations.js

@@ -43,13 +43,21 @@ function isExpectedValidationError(errorResult) {
 
 function isExpectedCase(errorResult, w3cManifestInfo) {
   const errorParams = errorResult.member.split('/').slice(1) //example: /icons/0/purpose -> ['icons', '0', 'purpose']
-  const isIconsPurpose = errorResult.member.includes("icons") && errorResult.member.includes("purpose")
+  const isIconsPurpose = errorResult.member.includes("icons") && errorResult.member.includes("purpose");
+  const isShortcuts = errorResult.member.includes("shortcuts");
+  const isScreenshots = errorResult.member.includes("screenshots"); // in the case we need to extend it to screenshots as well...
 
-  if (isIconsPurpose) {
+  if ((isScreenshots || isShortcuts) && isIconsPurpose) {
+    // don't handle, just let it go through
+    return true;
+
+  } else  if (isIconsPurpose) {
     const [icons, index, purpose] = errorParams
 
-    return w3cManifestInfo.content[icons][index][purpose].split(" ").filter(entry => {
-      return (entry === "any" || entry === "maskable" || entry === "monochrome");
+    return w3cManifestInfo.content.icons
+      && w3cManifestInfo.content.icons[index]
+      && w3cManifestInfo.content.icons[index].purpose.split(" ").filter(entry => {
+        return (entry === "any" || entry === "maskable" || entry === "monochrome");
     }).length > 0
   }
 

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "pwabuilder-lib",
-  "version": "2.1.9",
+  "version": "2.1.10",
   "description": "PWA Builder Core Library",
   "repository": {
     "type": "git",