🇳🇱Netherlands @jaapjansma

The merge request looks good to me. I have merged it. Thanks

There is no reason for having one handler per webform. Although I cannot see a usecase where one wants to have multiple handlers, I do think that makes the setup on the Drupal side probably more complex than needed.

Okay. I see. Thanks I have merged the MR! Thanks a lot

This seems to be already fixed in the 2.1.x version of the module.

Thanks

Thanks. I am not sure about the change you made on line 219. Is that one on purpose? Or is that an accidental change?

This might be solved meanwhile. Can you test again?

This is now added in version 2.1.0-beta8 of this module. You have to make sure that name the elements in your form processor as 'address_city', 'address_country' etc... The address_ is the key of the composite element. The _city and _country are the sub elements (which you cannot give another name in the webform.

@dnt can you create a merge request with the proposed fix?

jaapjansma → made their first commit to this issue’s fork.

jaapjansma → created an issue.

jaapjansma → created an issue.

I have merged your proposed fix. Thanks!

Thanks!

I could not reproduce this memory error. However I did found another issue whilst installing this module. I have fixed that issue.

Thanks @aman_lnwebworks! I have merged it.

Fixed

This is fixed for a while now.

jaapjansma → created an issue.

I can confirm that #7 fixes the issue.

Thanks. The changes are merged into the source code.

The changes from this issue already pushed by me with another commit.

jaapjansma → made their first commit to this issue’s fork.

jaapjansma → made their first commit to this issue’s fork.

jaapjansma → created an issue.

Klaas → credited jaapjansma → .