Skip to content

fix(site): Upload template files on template version editor#6222

Merged
BrunoQuaresma merged 6 commits into
mainfrom
fix/upload-template-files
Feb 16, 2023
Merged

fix(site): Upload template files on template version editor#6222
BrunoQuaresma merged 6 commits into
mainfrom
fix/upload-template-files

Conversation

@BrunoQuaresma

@BrunoQuaresma BrunoQuaresma commented Feb 15, 2023

Copy link
Copy Markdown
Contributor

How to QA:

  • Go to any template, preferably a template you can break.
  • At the bottom, select a version
  • Click on the edit button or add /edit at the end of the URL
  • Edit any file, preferably with a comment, and build it
  • If that works as expected - with a success message - the goal was achieved.
  • You can also navigate back to the template page and access the latest version created by you, click on edit, and check if all the changes you made are still there.

Strategy

  • The strategy was to append all the non-supported files in the tar first and after, add the ones from the editor file tree. Why? By doing that, we can rely on the file tree to append or not the editable files.
  • I'm also planning to remove untar dependency to use the util we have, but probably it is also going to be done in a next PR

Testing
Instead of testing the entire flow, I can try to extract the "update template version files" to test only this in isolation. I don't know how hard is but it sounds valuable. I'm probably going to do that in the next PR.

@BrunoQuaresma

Copy link
Copy Markdown
Contributor Author

I tested:

  • Create a new file
  • Create a new folder
  • Delete a folder
  • Delete a file

It is broken:

  • Rename is broken but I'm working to fix that on this PR or in the next one since are different issues

@BrunoQuaresma

Copy link
Copy Markdown
Contributor Author

Fix #6217

Comment thread site/js-untar.d.ts
@BrunoQuaresma BrunoQuaresma merged commit 909fbb6 into main Feb 16, 2023
@BrunoQuaresma BrunoQuaresma deleted the fix/upload-template-files branch February 16, 2023 16:59
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants