palkan / view_component-contrib

A collection of extension and developer tools for ViewComponent
MIT License
369 stars 22 forks source link

If Webpack is not being used, user is not asked if they are using Stimulus; no index.js files are created by custom generator #17

Open unikitty37 opened 2 years ago

unikitty37 commented 2 years ago

What did you do?

Created a Rails app using the template.

What did you expect to happen?

I should have been asked if I used Stimulus, and the custom generator should have created the relevant .js files.

What actually happened?

I was not asked if I used Stimulus, because all the USE_STIMULUS code is wrapped in an if USE_WEBPACK block; I instead got a reference to #14.

Additional context

It would be better to ask the user if they're using Stimulus in all cases, then install the relevant config if Webpack is being used, otherwise give the reference to #14.

As a result, the custom generator does not have an option to create index.js, and the user must do it themselves each time.

Environment

Ruby Version: ruby 3.1.2p20 (2022-04-12 revision 4491bb740a) [aarch64-linux]

Framework Version (Rails, whatever): Rails 7.0.3.1

View Component Contrib Version: view_component-contrib (0.1.1)

palkan commented 2 years ago

Thanks for the report!

Agree, we should ask a user if they want to add Stimulus-related stuff to generators independently of whether Webpack is being used. Would you like to prepare a PR?

helpotters commented 1 year ago

Would you like me to tackle this?

palkan commented 1 year ago

Hey @helpotters! Yes, please (if you're still interested)

helpotters commented 1 year ago

I'll check it out this weekend!