drush icon indicating copy to clipboard operation
drush copied to clipboard

Fixes #3793 Can not create new config via cset

Open geek-merlin opened this issue 6 years ago • 6 comments

geek-merlin avatar Jul 05 '19 21:07 geek-merlin

Sorry for 2 year delay!

This makes sense, but on the other hand its handy for most users to validate this. So, I'm thinking we keep validation on but prompt to ask user if we should proceed. That prompting should be done in cset itself, and not the validation (i.e. handle the validation exception).

weitzman avatar Jun 11 '21 13:06 weitzman

Thanks for looking into this! Unfortunately i won't be able to work on this any soon.

geek-merlin avatar Jun 11 '21 23:06 geek-merlin

That prompting should be done in cset itself, and not the validation (i.e. handle the validation exception).

That wont work. Needs more thought.

weitzman avatar Jun 20 '21 12:06 weitzman

Could you create a yaml file outside of drush and then do config:import --partial?

weitzman avatar Jan 19 '22 13:01 weitzman

(config:import --partial) Thanks! But: That would be no win compared to "edit config-export and cim".

Your plan to use interaction sounded reasonable and i did not get why it's not feasible.

Otoh why not simply create? Typos? We may add a message "Config created - or was it a typo?" so users who created config due to a typo can cdel then. No kittens harmed.

geek-merlin avatar Jan 19 '22 18:01 geek-merlin

May I add a respectful +1 for @geek-merlin's suggestion just to create the config object please @weitzman?

As far as I can tell, a mis-typed configuration name won't do any harm to the site.

Best wishes,

Alex

alexharriesredactive avatar Feb 22 '22 10:02 alexharriesredactive

this got done in drush 12 (releasing this week)

weitzman avatar May 28 '23 13:05 weitzman