windyScripts / dev-directory

21 stars 5 forks source link

#115 create skills user skills interest user interest tables #128

Open jmoldyvan opened 1 year ago

jmoldyvan commented 1 year ago

Closes

115

Description

Added new models and migration. Models created: Skills, Interests, UserSkills, UserInterests Updated server/models/index.ts to reflect new models Created new migrations and ran migrations to include the 4 new tables based off of the models.

Testing

Run test. Lint. Made sure everything was spelled correctly. Run migrate and check database to have the tables (Skills, Interests, UserSkills, UserInterests) with the correct attributes as described in #115

Type of change

New feature (non-breaking change which adds functionality)

Screenshots

Please include any screenshots if there are visual changes.

Checklist:

[X] Changes have new/updated automated tests, if applicable
[X] Changes have new/updated docs, if applicable
[X] I have performed a self-review of my own code
[X] I have added comments on any new, hard-to-understand code
[X] Changes have been manually tested
[X] Changes generate no new errors or warnings

windyScripts commented 1 year ago

What I see looks good. I'm unable to find the Interests and UserInterests tables after running the migration, and I'm not sure if I've made an error.

Altering the casing to snake_case would be in keeping with design decisions we've made as well.

devonzara commented 1 year ago

@jmoldyvan, how's this going? Anything we can do to help get this approved and merged?