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.

Sign up for GitHub

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

Jump to bottom

#36 Fixes bug for allowing plural word schemas #38

Merged
merged 2 commits into from
Dec 18, 2018
Merged

#36 Fixes bug for allowing plural word schemas #38

merged 2 commits into from
Dec 18, 2018

Conversation

Cleop
Copy link
Member

@Cleop Cleop commented Dec 17, 2018 •
edited
Loading

#36 Fixes bug for allowing plural word schemas

@Cleop Cleop self-assigned this Dec 17, 2018
@Cleop Cleop changed the title WIP - #36 Fixes bug for allowing plural word schemas #36 Fixes bug for allowing plural word schemas Dec 18, 2018
@Cleop Cleop assigned nelsonic and unassigned Cleop Dec 18, 2018
@Cleop Cleop requested a review from nelsonic December 18, 2018 11:46
nelsonic
nelsonic reviewed Dec 18, 2018
View reviewed changes
test/support/test_autoform/lib/test_autoform/sample_product.ex
use Ecto.Schema
import Ecto.Changeset

schema "products" do
schema "sample_products" do
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

nelsonic
nelsonic approved these changes Dec 18, 2018
View reviewed changes
Copy link
Member

@nelsonic nelsonic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Cleop 🎉

@nelsonic nelsonic merged commit f02e2d7 into master Dec 18, 2018
@nelsonic nelsonic deleted the plural-word-schema-#36 branch December 18, 2018 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nelsonic nelsonic nelsonic approved these changes

Assignees

@nelsonic nelsonic

Labels
awaiting-review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Cleop @nelsonic