Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Conditional options on config.json #390

Closed
rasouza opened this issue Aug 10, 2020 · 2 comments · Fixed by #454
Closed

Conditional options on config.json #390

rasouza opened this issue Aug 10, 2020 · 2 comments · Fixed by #454
Assignees
Labels
✨ feature Suggest a new feature or enhancement to the Ritchie project
Milestone

Comments

@rasouza
Copy link

rasouza commented Aug 10, 2020

What would you like to be added:
I'd like to show an option based on the answer of the previous option
Why is this needed:
So I can deliver a better experience to the user by removing unnecessary questions depending on his/her answer

@rasouza rasouza added the ✨ feature Suggest a new feature or enhancement to the Ritchie project label Aug 10, 2020
@victor-schumacher
Copy link
Contributor

@rasouza that sound's nice! Thanks for the idea.
Today cli reads inputs from config.json on this file:

func (in InputManager) fromPrompt(cmd *exec.Cmd, setup formula.Setup) error {

The implementation of conditional options will probably go there.
Pull requests about this feature are welcome.

@henriquemoraeszup
Copy link
Contributor

Hey @rasouza ! thanks for your suggestion. Your request is currently under review on #454.
Stay calm, and automate everything 👊

@kaduartur kaduartur changed the title [FEATURE] Conditional options on config.json Conditional options on config.json Aug 28, 2020
@kaduartur kaduartur added 🚀 QA and removed ✔️ ready-for-review ready for review labels Aug 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
✨ feature Suggest a new feature or enhancement to the Ritchie project
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants