Replies: 2 comments
-
This is a great callout. Thanks! Tentative fix here: #63 would love your eyes on it if you have a sec. |
Beta Was this translation helpful? Give feedback.
0 replies
-
PR merged! Happy to refine further, too. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I tried using Hotwire Combobox this morning and hadn't implemented the
to_combobox_display
method in my model. I got an error and it took me a minute to understand what I needed to do. It would be nice to have a better exception raised here.This method may need a refactoring to provide different error messages for each usage:
hotwire_combobox/lib/hotwire_combobox/helper.rb
Lines 121 to 127 in d171cb9
It would be really helpful to new users to provide some thorough error messages in these cases, for example with the above missing method:
Beta Was this translation helpful? Give feedback.
All reactions