Bind RecipeForm.changeAction to the class instance. #239
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I noticed locally that the recipe form wasn't showing the action forms when I changed the action when creating a new recipe, and tracked it down to not binding this function properly. The dev server doesn't have this problem, weirdly. Not sure why I'm only seeing it locally.
I wanted to create a test for this, but the right test IMO is to test that the console-log form is shown when changing the select box via a user interaction, and we don't really have a great setup yet for running those kind of tests. It's also awkward to this via unit tests because the component as written is generic, so a test that relies specifically on console-log seems to not fit the unit tests for the component.
I'm open to feedback on better ways to test this. Otherwise, r?