Feature technology table

[brdeleon]

Overview

Fixes #39

Action Items

• add issue link to the tables tab of the spreadsheet

• identify and document table description (see spreadsheet under Resources)

• compare and check off the data fields (below) against the ERD. Note any fields in the ERD that's missing from the list. (see Resources)

  • Review and correct discrepancies

• compare the check off associated tables (below) against the ERD. Note any associated tables in the ERD that's missing from the list (see Resources)

• create a single model in Django (defining schema)

• write a test for the relationships this model will have with other models (e.g., creating a user and assigning them a set of permissions on a project).

• write an API end point

• write API unit tests

• document the endpoint

• Release Create Table: project_technology_xref #40 from ice box

[fyliu]

Everything works well! The rest are code cleanup changes that don't affect functionality.

I'm listing them here.

1. There's a few lint warning that are supposed to be automatic fixes. They're whitespaces and import reorderings. You can install pre-commit to try to do that or I can just commit them. There's a pre-commit section in the contributing page. These commands are useful to know:

   # run lint, which is a smaller set of checks than the pre-commit hooks, but doesn't require installing pre-commit
   ./scripts/lint.sh
   
   # the commands below require installing pre-commit
   # run individual pre-commit hooks
   pre-commit run end-of-file-fixer -a
   pre-commit run trailing-whitespace -a
   
   # run all the precommit hooks
   ./scripts/precommit-check.sh

2. The migration files can be combined into one since the first 2 cancel out. You can do that with these steps:

   1. Roll back to before the first new migration

      docker-compose exec web python manage.py migrate core 0012

   2. Delete all the new migration files

      rm app/core/migrations/{0013*,0014*,0015*}

   3. Rerun make migrations and migrate

      docker-compose exec web python manage.py makemigrations
      docker-compose exec web python manage.py migrate

[chelseybeck]

Looks like you made the requested changes. Great work.

requested changes were completed