Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating readme schema example #13

Merged
merged 2 commits into from
Nov 11, 2016
Merged

Updating readme schema example #13

merged 2 commits into from
Nov 11, 2016

Conversation

mahcloud
Copy link
Contributor

Using separate params schema module for custom ecto validations on api params.
Awesome product! Exactly what we needed. 👍

I noticed two issues with the example in the readme for this.

  • from function's second parameter is a list. So, the current example gives an function not found error because it doesn't pattern match because it is expecting a list.
  • Ecto.Changeset is not aliased so it gives a function not found error on cast and validate methods.

@vic vic merged commit bbebc09 into vic:master Nov 11, 2016
@vic
Copy link
Owner

vic commented Nov 11, 2016

@mahcloud thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants