Clean up and docs

src/react/components/pages/editorPage/editorPage.tsx

@@ -308,6 +308,11 @@ export default class EditorPage extends React.Component<IEditorPageProps, IEdito
         this.renameTagConfirm.current.open(tagName, newTagName);
     }
 
+    /**
+     * Renames tag in assets and project, and saves files
+     * @param tagName Name of tag to be renamed
+     * @param newTagName New name of tag
+     */
     private onTagRenamed = async (tagName: string, newTagName: string): Promise<void> => {
         const { project, selectedAsset } = this.state;
         const newProject = {
@@ -332,6 +337,10 @@ export default class EditorPage extends React.Component<IEditorPageProps, IEdito
         this.deleteTagConfirm.current.open(tagName);
     }
 
+    /**
+     * Removes tag from assets and projects and saves files
+     * @param tagName Name of tag to be deleted
+     */
     private onTagDeleted = async (tagName: string): Promise<void> => {
         const { project, selectedAsset } = this.state;
         const newProject = {

src/services/assetService.test.ts

@@ -369,7 +369,6 @@ describe("Asset Service", () => {
                         },
                     ],
                 ),
-
             };
 
             const project = populateProjectAssets();
@@ -422,8 +421,8 @@ describe("Asset Service", () => {
                         },
                     ],
                 ),
-
             };
+
             const project = populateProjectAssets();
             const assetService = new AssetService(project);
             await assetService.renameTag(project.assets, tag1, newTag, assetMetadata);

src/services/assetService.ts

@@ -234,9 +234,11 @@ export class AssetService {
      * Update tags within asset metadata files
      * @param assets The assets containing tags to update
      * @param tagName Name of tag to update within project
+     * @param transformer Function that accepts array of tags from a region and returns a modified array of tags
      * @param currentAsset Current asset being viewed. Makes changes and returns updated asset to avoid
      * needing to reload the asset in the editor page
-     * @param transformer Function that accepts array of tags from a region and returns a modified array of tags
+     * @returns Modified `currentAsset`. Returns `null` if asset did not need to be modified 
+     * or if `currentAsset` is null or undefined
      */
     private async updateAssetTags(
             assets: {[id: string]: IAsset},