clay-kiln icon indicating copy to clipboard operation
clay-kiln copied to clipboard

Validate props, 🔪 all args

Open rmfarrell opened this issue 6 years ago • 1 comments

Bug

I think the currentargs is overloaded and doesn't validate incoming data. This causes misconfigured yaml files from clay to fail silently or without really helpful error messages, and it makes the kiln internal code a bit hard to reason.

Proposed Solution/Implementation

I think we should stop using this prop in favor of passing property values to the component individually. Vue has built-in property validation right at the component level, where you'd want it.

rmfarrell avatar Aug 17 '18 19:08 rmfarrell

merging this into https://github.com/clay/clay-kiln/issues/1310

nelsonpecora avatar Aug 28 '18 15:08 nelsonpecora