fix: when add remote skill show manifest json parse error #7396

zhixzhan · 2021-04-28T00:10:35Z

Hi @cwhitten , I made some update base on your change here . Is this your intention?

Yes thank you @zhixzhan !

-Original file line number
+Diff line change
@@ Expand Up @@
         });
         setSkillManifest(data);
       } catch (error) {
-        setFormDataErrors({ ...error, manifestUrl: formatMessage('Manifest URL can not be accessed') });
+        const httpMessage = error?.response?.data?.message;
+        const message = httpMessage?.match('Unexpected string in JSON')
+          ? formatMessage("Error attempting to parse Skill manifest. There could be an error in it's format.")
+          : formatMessage('Manifest URL can not be accessed');
+        setFormDataErrors({ ...error, manifestUrl: message });
       }
     };
     const getTriggerFormData = (intent: string, content: string): TriggerFormData => ({
@@ Expand Down @@

Provide feedback