Skip to content
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

chore(API): refactor template upload #1181

Merged
merged 3 commits into from
Dec 20, 2024

Conversation

open-dynaMIX
Copy link
Member

@open-dynaMIX open-dynaMIX commented Dec 19, 2024

Since the refactoring of the build, the documented commands don't work anymore. Additionally the path to the templates changes now dynamically with the python version used. This commit puts all the templates into one global template directory. The command is refactored, so that it's sufficient to provide the file name. The path to the template will then be constructed in the command.

More commits:

feat(API): allow for uploading multiple templates at once
This commit refactors the upload_template command, so that is now uses
positional arguments for templates and can handle multiple at once.

chore: improve dev docs

Since the refactoring of the build, the documented commands don't work
anymore. Additionally the path to the templates changes now dynamically
with the python version used. This commit puts all the templates into
one global template directory. The command is refactored, so that it's
sufficient to provide the file name. The path to the template will then
be constructed in the command.
@open-dynaMIX open-dynaMIX force-pushed the refactor_template_upload branch from 73db40d to 13560b3 Compare December 19, 2024 15:28
This commit refactors the `upload_template` command, so that is now uses
positional arguments for templates and can handle multiple at once.
@open-dynaMIX open-dynaMIX merged commit 8dc67f9 into adfinis:main Dec 20, 2024
6 checks passed
@open-dynaMIX open-dynaMIX deleted the refactor_template_upload branch December 20, 2024 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants