no spurious error on asset editor response

microsoft · Sep 19, 2024 · fbd8456 · fbd8456
1 parent fe3898c
commit fbd8456
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/localtypings/pxteditor.d.ts b/localtypings/pxteditor.d.ts
@@ -1296,6 +1296,7 @@ declare namespace pxt.editor {
 
     interface BaseAssetEditorResponse {
         id?: number;
+        success: boolean;
     }
 
     interface OpenAssetEditorResponse extends BaseAssetEditorResponse {

diff --git a/webapp/src/assetEditor.tsx b/webapp/src/assetEditor.tsx
@@ -58,7 +58,8 @@ export class AssetEditor extends React.Component<{}, AssetEditorState> {
 
                 this.sendResponse({
                     id: request.id,
-                    type: request.type
+                    type: request.type,
+                    success: true
                 });
                 break;
             case "open":
@@ -73,7 +74,8 @@ export class AssetEditor extends React.Component<{}, AssetEditorState> {
                 });
                 this.sendResponse({
                     id: request.id,
-                    type: request.type
+                    type: request.type,
+                    success: true
                 });
                 break;
             case "duplicate":
@@ -85,14 +87,16 @@ export class AssetEditor extends React.Component<{}, AssetEditorState> {
                 });
                 this.sendResponse({
                     id: request.id,
-                    type: request.type
+                    type: request.type,
+                    success: true,
                 });
                 break;
             case "save":
                 this.sendResponse({
                     id: request.id,
                     type: request.type,
-                    files: this.saveProjectFiles()
+                    files: this.saveProjectFiles(),
+                    success: true
                 });
                 break;
         }