search

ezsystems / launchpad

CLI tool to bootstrap an eZ Platform project Docker stack | #justcode
https://ezsystems.github.io/launchpad
MIT License
22 stars 27 forks source link

set default answer to true for http basic if composer auth is needed #22

Closed crevillo closed 6 years ago

crevillo commented 6 years ago
Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Fixed tickets

When users says the want the options standard-with-http-auth, default anwser for the question should be yes instead of no.

i've seen the condition 

if (!$this->requireComposerAuth()) {
      return [];
}

which makes the function return is 0 was selected at first place. now, when 1 is selected (standard with-composer-auth) the default answer will be yes, while if mode expert is selected, it will be false as it happens now.

@Plopix , now another question is... do we need to ask this question thinking that the users have already said they want this when they select 1 at first place?

Plopix commented 6 years ago

Thank you @crevillo! https://www.ihighfive.com/