Suggestion: move appform->inquirer conversion to appform class definitions?

omnivector-solutions / jobbergate-cli

Jobbergate CLI

MIT License

2 stars 0 forks source link

Suggestion: move appform->inquirer conversion to appform class definitions? #66

Open NilesDev opened 3 years ago

NilesDev commented 3 years ago

Logic for converting an appform. object into inquirer. object is now in jobbergate_api_wrapper (JobbergateApi.assemble_questions) Moving this to their respective appform classes could free up some space in the somewhat bulky wrapper.

zsinx6 commented 3 years ago

We already created a task in monday for this, there will be also some other improvements for the cli, since the new api is more flexible and simpler

jamesbeedy commented 3 years ago

@NilesDev suggestion welcome, and thanks for making some noise here. As @zsinx6 mentioned, this has been added to the list of upcoming work. Thanks!