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

#166151268: Add an author foreign key to exercises #29

Merged
merged 3 commits into from
Aug 2, 2019

Conversation

willeswa
Copy link
Contributor

What does this PR do?

Adds a foreign key to exercise that points to the user

Description of Task to be completed?

  • Add tests for the above implementation
  • Implement the above task

How should this be manually tested?

  • Run runserver and create a new exercise
  • In the database, you should see an author_id column referencing the use

What are the relevant pivotal tracker stories?

#166151268

Screenshots

None

@willeswa willeswa changed the title 166151268-feat(models): add an author foreign key to exercises #166151268: Add an author foreign key to exercises Jun 20, 2019
- add a foreign key field to the models
- write a test to assert that relation exists

[Delivers #166151268]
@willeswa willeswa force-pushed the ft-author-exercise-fk-166151268 branch from f6442b5 to 407a550 Compare June 20, 2019 18:34
@ceexon ceexon force-pushed the ft-author-exercise-fk-166151268 branch 3 times, most recently from 90498d4 to 4712cb6 Compare August 1, 2019 15:28
- add a foreign key field to the models
- write a test to assert that relation exists

[Delivers #166151268]
@ceexon ceexon force-pushed the ft-author-exercise-fk-166151268 branch from 4712cb6 to 9968b1d Compare August 1, 2019 15:56
@wekesa931 wekesa931 force-pushed the ft-author-exercise-fk-166151268 branch from 27edf1f to 8745f90 Compare August 2, 2019 08:06
@wangonya wangonya merged commit 32451b4 into develop Aug 2, 2019
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