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

change primary key field #15

Closed
CriztianiX opened this issue Oct 29, 2014 · 4 comments · Fixed by #28
Closed

change primary key field #15

CriztianiX opened this issue Oct 29, 2014 · 4 comments · Fixed by #28
Labels
enhancement

Comments

@CriztianiX
Copy link

Hello... i have the issues what my db don't have a primary key called id.... theirs name are.. "brand_id", "city_id", etc.... how i can specify it?
@ostinelli
Copy link
Owner

You currently can't. Added to the enhancement requests.

@CriztianiX
Copy link
Author

oh good... i wait for this!

@istr
Copy link
Contributor

istr commented Nov 5, 2014

You may want to have a look at my fork. I implemented {tablename}_id or id logic there. I did not yet manage to put it through proper qa (test suites etc), however. Will send a pull request if this is done.

@ostinelli
Copy link
Owner

Thank you, will wait for the pull request then.

istr added a commit to istr/gin that referenced this issue Sep 9, 2015
@istr
try to address issue ostinelli#15
e0c8750
@istr istr mentioned this issue Sep 9, 2015
Merged
istr added a commit to istr/gin that referenced this issue Sep 9, 2015
@istr
Try to address issue ostinelli#15
9a6394c
ostinelli added a commit that referenced this issue Sep 14, 2015
@ostinelli
Merge pull request #28 from istr/table-id
cb35e87 
Allow models to define a different primary key field (issue #15).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Try to address issue #15 istr/gin
3 participants
@ostinelli @CriztianiX @istr