Add ability to download config #69
Conversation
make json gen better
|
@thenetworkgrinch I think this is done, if you want to check it out, that'd be great. |
|
Oh! Glhf then, hope it goes well! |
|
Finally had a chance to review this, sorry for taking so long. I love it! The only thing I wonder if it is necessary is the () in onclick which is semantic at best and not really a concern. I love how you used the namespace to bypass popup blockers too! |
|
The only other semantic things I will fix at a later date are you usage of semicolon's but I dont think I was consistent with those either so it is a problem for later. |
|
Lol idk about all that, I'm just getting into web dev, coming from Java python and c# and thought I'd give this a go (google is my best friend). Also I didn't write the saveAs function, it's credited above it as a drop in replacement for a library that wasn't working. |
Generates a zip of the generated json config in the correct file structure, and serves it to the user. IMO a Useful QOL Change that eliminates more of the configuration headache.