Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Feb 16, 2023

Conversation

BrunoQuaresma
Copy link
Collaborator

@BrunoQuaresma BrunoQuaresma commented Feb 15, 2023

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
Collaborator 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
Collaborator Author

Fix #6217

@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