search

Nebo15 / ecto_mnesia

Ecto adapter for Mnesia Erlang term database.
https://hex.pm/packages/ecto_mnesia
MIT License
242 stars 41 forks source link

handle uuid #69

Open cmedley opened 6 years ago

cmedley commented 6 years ago

fixes ** (RuntimeError) trying to load string UUID as Ecto.UUID when using :binary_id

AndrewDryga commented 6 years ago

Hello, @cmedley! This is your first Pull Request that will be reviewed by Ebert, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information.

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 88.74% when pulling 70516104f4cc4db036318de913fd6e9efa454595 on cmedley:handle-uuid into de6ef5f991e838fa2a11d706b9791f451231ce7f on Nebo15:master.

AndrewDryga commented 6 years ago

@cmedley good catch, can you please add a test case so we won't have a regression in future?