Skip to content

Create table leadership type 42 #491

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

dmartin4820
Copy link
Member

Fixes #42

What changes did you make?

  • create leadership_type model
  • register leadership_type in admin
  • add leadership_type endpoints
  • add leadership_type seed data
  • add tests for leadership_type creation
  • add test for leadership_type to project relationship

Why did you make the changes (we will use this info to test)?

  • Here the seed data has an id column with blank id fields. This creates an empty column in the resulting .csv file used for creating seed data. To complete adding the seed data, I manually removed this column from the .csv file after dowloading from the Google sheet.
  • Uncommented the leadership_type foreign key here. This is done so that the relationship is testable.

@dmartin4820 dmartin4820 requested a review from del9ra May 5, 2025 01:28
@github-project-automation github-project-automation bot moved this to PR Needs review (automated column, do not place items here manually) in P: PD: Project Board May 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: PR Needs review (automated column, do not place items here manually)
Development

Successfully merging this pull request may close these issues.

Create Table: leadership_type 😄
1 participant