aerni / statamic-livewire-forms

Supercharge your Statamic forms with Livewire
https://statamic.com/addons/aerni/livewire-forms
Other
28 stars 2 forks source link

Form field names #10

Closed stuartcusackie closed 3 years ago

stuartcusackie commented 3 years ago

Hi,

This refers to this closed issue: https://github.com/aerni/statamic-livewire-forms/issues/6

It's great that form inputs now have unique ids but I was assuming the input names would stay the same. Screenshot:

image

The problem is my logic relies on certain field names being passed such as email, but with the update it's hard to target because now it's called JqcAhRBDtUd3sDwcJgiq_email

I can probably work around this but do you think it would be better to keep the field names as they were? There's no harm in having duplicate field names, it's just duplicate field ids that produce the console warnings.

UPDATE: Ignore for now. This may not be an issue. I will be back soon...

stuartcusackie commented 3 years ago

Yep, my bad. This still works fine with my code.

Closing.