Add support for serial ids in HasMany field

[thomasfortes]

Add a match function that handles when option.id is represented as an integer in the HasMany field.

Closes #451 ?

[Flo0807]

String.to_integer may throw an argument error. Do you see any problems with converting the integer to a string instead of the other way around?

Like this:

defp match_option_to_id(option, id), do: to_string(option.id) == to_string(id)

@thomasfortes

[thomasfortes]

It works just fine with to_string and is even simpler because we can remove the functions and just change inside the handle_event.

The only thing that I can think of is that when the internal representation is guarded to be an integer and String.to_integer fails it probably means that something else is wrong and knowing it earlier is better than letting it fail silently.

What do you think?

[Flo0807]

The only thing that I can think of is that when the internal representation is guarded to be an integer and String.to_integer fails it probably means that something else is wrong and knowing it earlier is better than letting it fail silently.

True! Let's try it with your solution 👍