-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Add a way to export recipes as JSON #15383
Comments
This might be useful if the recipe size is too big |
The file download zip file includes recipe json content. It's it possible to read the json from zip - it's just need to be updated in document. |
I think Mike's intend to simplify the process by downloading the JSON instead of zip then extracting the |
Yes but extracting from zip is easy. Downloading zip file vs json could be confusing as both are not different. Better option to improve existing file download with option to compress the file or not |
I'm afraid I have to disagree this will let every target be compressed / uncompressed, a good option is to add a checkbox, so when it is checked the file will be compressed regardless of its format JSON, XML .. etc |
First of all, deployment only support JSON format. We do not support XML or any other format. So complicating usage/design just because it not a good idea. I do not mind the idea of adding a checkbox to compress or not. But this will require to change lots of our existing setup. We'll need to change how we render the target to use shapes, also add new templates for adding the checkbox. I don't think we need to overhaul our setup just to allow adding a checkbox. The current structure allow us to extent the UI by adding a new deployment source easily which is the approach I took. |
I know, perhaps some of the clients already have
Checkbox could be a good option IMHO regardless of the code change |
The fact that OC supports only JSON recipes, we don't care about other formats. I don't think it's a valid argument.
If you have the time, feel free to submit a PR converting deployment rendering to shapes and add the checkbox. My intention with the submitted PR to utilize the structure that already exists "path of least resistance". Also, changing the structure could also mean possible breaking change people. Implementing a battleship when we only need a boat is an overkill ;) |
Instead of having two targets Can we have single target File Download with two action with drop-down button? Default will be compressed, second will be JSON? |
@ns8482e check out the my last comment I do not want to reconstruct the entire setup. Deployment sources are open for extension using cards. I am looking into an option of adding property that would allow us to expose compressed or not. |
I don't think you need to reconstruct the entire setup, shape could help here. @MikeAlhayek thanks IMHO only one target |
@ns8482e okay. How about this? |
Much better, that's why I prefered the checkbox to avoid adding a dropdown on each target |
Looks good now |
Many times we export small recipes. We don't really want to download it as compressed then uncompress it.
It would be much easier if we add an option to export recipes as json.
The text was updated successfully, but these errors were encountered: