tee8z / doppler

A tool for building out and managing bitcoind/lightning clusters for regtest or signet environments
Creative Commons Zero v1.0 Universal
45 stars 7 forks source link

Add Blockly Editor for Doppler Files #33

Closed thebrandonlucas closed 1 year ago

thebrandonlucas commented 1 year ago

https://github.com/tee8z/doppler/assets/38222767/fd5a374d-e5f5-4614-87c5-6baa24ef68c6

Future work:

tee8z commented 1 year ago

This is a great idea and looks look a solid start! Another suggestion is to simplify the language on the blocks or to add symbols to them, since we're adding another layer we can use this to abstract away more of the complexity from the user unless they want to deal with it.

thebrandonlucas commented 1 year ago

This is a great idea and looks look a solid start! Another suggestion is to simplify the language on the blocks or to add symbols to them, since we're adding another layer we can use this to abstract away more of the complexity from the user unless they want to deal with it.

Definitely agree! Will look at simplifying in the next PR. There are also other ways we can do this if we want further clarity. We could add "fields" that make it clearer but a bit more cumbersome to use. For example, we could have them look like below, which is probably better for clarity but a bit more cumbersome to use if you already know what you're doing. We could do either of these styles below as alternatives, let me know which you prefer:

Screenshot 2023-09-16 at 9 09 31 AM Screenshot 2023-09-16 at 9 11 41 AM

Also, I don't have permissions to merge this PR so feel free to merge if you think it's ready!

tee8z commented 1 year ago

This is a great idea and looks look a solid start! Another suggestion is to simplify the language on the blocks or to add symbols to them, since we're adding another layer we can use this to abstract away more of the complexity from the user unless they want to deal with it.

Definitely agree! Will look at simplifying in the next PR. There are also other ways we can do this if we want further clarity. We could add "fields" that make it clearer but a bit more cumbersome to use. For example, we could have them look like below, which is probably better for clarity but a bit more cumbersome to use if you already know what you're doing. We could do either of these styles below as alternatives, let me know which you prefer: Screenshot 2023-09-16 at 9 09 31 AM Screenshot 2023-09-16 at 9 11 41 AM

Also, I don't have permissions to merge this PR so feel free to merge if you think it's ready!

I think I like the first style more than the long line one, but feel free to play around and see what feels easier to use, willing to be wrong.