Luke Towers
Luke Towers
@mjauvin do you feel like having a play around with this to help test it?
> You all know the love I have for npm... why do you even ask? 😂 That's why I ask 😂
@jaxwilko does it make sense to add support to the AssetMaker trait as well?
@jaxwilko is there a docs PR?
Just waiting for @bennothommo to weigh in on the remaining two items (package.json being excluded from the source export & moving the PackageJson class into either the Storm Parse package...
@jaxwilko @bennothommo are there any remaining items before we can merge this?
Going to need unit tests for this (covering existing behaviour and the change) before this can be merged.
@bennothommo our specific use case is showing a preview of the field in a visual formbuilder so we would want all options to be displayed.
Whats the status on this @jaxwilko @bennothommo?
I'm fine with merging this and if it becomes a problem in the future that too many options are shown on the preview screen then a `hideWhenReadonly` or similar option...