search

gjaldon / ecto_enum

Ecto extension to support enums in models
MIT License
562 stars 131 forks source link

Implement string backed enums. #9

Closed mtwilliams closed 5 years ago

mtwilliams commented 8 years ago

This is in response to #4.

Two points:

1) I've tried to maintain (or improve) test coverage. 2) This doesn't perform sanity checks in load – instead relying on Ecto to do the right thing.

I feel my additions might not align with your style, please let me know what to fix.

gjaldon commented 8 years ago

@mtwilliams btw, you'll need to rebase since I've made changes.

JonnyPower commented 8 years ago

What's the status with this?

mtwilliams commented 8 years ago

@JonnyPower

I ended up hard forking for reasons relevant to our intervals @meetwalter.

I don't currently have the time to restructure the changes and open the chain of pull requests.

cesariouy commented 7 years ago

These changes have been included in #50 (thanks @mtwilliams!)

gjaldon commented 5 years ago

This is stale since we already support string-backed enums after the refactor