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

Update relationships.rst #83

Merged
merged 1 commit into from
Jul 10, 2015
Merged

Update relationships.rst #83

merged 1 commit into from
Jul 10, 2015

Conversation

cisko3000
Copy link
Contributor

I think it should be Event.id instead of Location.id in the Location model in one-to-many example.

@kvesteri
Copy link
Owner

Thanks for the PR! 🚋

kvesteri added a commit that referenced this pull request Jul 10, 2015
@kvesteri
Merge pull request #83 from 8oh8/patch-1
91bef18 
Update relationships.rst
@kvesteri kvesteri merged commit 91bef18 into kvesteri:master Jul 10, 2015
@cisko3000
Copy link
Contributor Author

What does PR stand for?
On Jul 10, 2015 11:15 AM, "Konsta Vesterinen" notifications@github.com
wrote:

Thanks for the PR! [image: 🚋]

—
Reply to this email directly or view it on GitHub
#83 (comment)
.

@kvesteri
Copy link
Owner

Pull Request :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@cisko3000 @kvesteri