Feature Request: Pass options to use:enhance

[ciscoheat]

Feature Request: Pass options to use:enhance

[ciscoheat]

Ah, good idea, will definitely implement that in a near release. Many of the options should be usable there. Any other options except the events that you may find interesting to use there?

[ciscoheat]

There is a fields property returned from superForm that arranges the form data as objects with separate name, errors, etc properties. It has some kinks though, but it version 0.6.0 it will be much improved.

The events I'm not sure what you mean exactly, you want to be able to add events after calling superForm? How does the default enhance handle it?

[ElijahJohnson5]

You are able to use enhance like

use:enhance={() => {
  do something before submit
  return () => {
    do something with result  
  }
}

Currently with super form you have to pass those options into the superForm method which makes it a little more difficult. If instead I could pass options to the superForm enhance method that would be very nice

[ciscoheat]

Removed the next label because it will take some consideration. Passing options in the enhance action will possibly conflict with the options set in superForm, and I'm not 100% sure how to handle that (overwrite, chain events together, etc?). Any ideas here are much appreciated.

[ElijahJohnson5]

I would think they would overwrite the options passed in the superForm but am open to either way

[ciscoheat]

I'm thinking about what would happen if the action is used in several places. Is that what you had in mind, btw? :) If so, I think event chaining would be the best option, it's too easy to cause surprises when calling it in several unrelated places, otherwise.

[ElijahJohnson5]

Yes multiple places is how I was/am using it. Chaining sounds good to me

[ciscoheat]

I think that will work fine, I'll start preparing that for the next release!

[ciscoheat]

@ElijahJohnson5 you should be able to pass events in use:enhance now:

<form method="POST" use:enhance={{onUpdated: ({form}) => ... }}>

Can you check that it works?

[ElijahJohnson5]

Yes it works great! Thank you

[ciscoheat]

Nice, closing this then. :)