Accept multiple attributes #30

[kirs]

No description provided.

[m-Peter]

That's great @kirs !!!

[kirs]

I'm glad you like it! AFAIK it's frozen now for Rails core team review as @kaspth suggested.

[m-Peter]

I'm currently reworking the ActiveForm::Base class. As you probably noticed, you have to provide the same methods for the ActiveForm::Base and ActiveForm::Form classes. Ideally the ActiveForm::Base should delegate such methods to the root model (which should be wrapped by a ActiveForm::Form class).

[kaspth]

@m-Peter You might want to check out the last commits in #18 more closely then 😁

My main realization was that main_model was really just another association and that there was no reason for it to be treated differently. I took to naming it ModelAssociation partly because a number of things had Form in their name and partly because the model being wrapped isn't really a form but an associated model 😉

Looking forward to what you come up with, 👍

[carlosantoniodasilva]

Unfortunately I don't think we should copy over Rails code to handle multiparameters, we should think of a way to extract that on the Rails side, but I'm not sure that's going to be done. I've proposed this extraction idea, lets see what others think first, if it's accepted we can start by that.