Conversation
|
It's a good idea :) Ideally this would use source.js to handle loading/saving of the preset data. |
|
Okay, I'll take a look at that. Thanks! |
|
I've now implemented both saving and loading of preset files. I used Let me know if there are other changes that you'd like to see. As I mentioned before, I haven't written any javascript for a very long time, and this is my first time touching Electron, so I'm sure I made some odd choices. |
|
Hey there, I'm not sure if @neauoire is still maintaining this, but feel free to make a PR to my fork of Pilot! I'm wanting to make a few improvements and update the dependencies and such, so maybe we can work there till this gets picked up again? https://github.com/jakofranko/Pilot |
|
Cool! I should have some time next month to take a look at this again. I haven't been using Pilot much recently, but I'd like to change that. |
|
Awesome! @ondras has already put his PR over there and I've updated to Electron v17. Let me know if you need another set of eyes, I'd be happy to take a look at your code if you run into issues. |
I'm planning on using Pilot for some new projects, and having the ability to save and load presets will be really useful for me, so I started trying to add the functionality.
So far I have basic preset loading working okay. As you can see, it simply loads Pilot commands from a text file and runs them. I haven't written any javascript for about 6 or 7 years, so please forgive me if I did anything weird. Happy to change anything about the implementation if you have any feedback.
Am planning on trying to add preset saving tomorrow, which I expect will be a bit more work. I thought it would be worth opening a PR now to start a conversation about this, and to see if you're even open to including this functionality in Pilot.
Please let me know what you think. Thanks!