HER 16: Migration -- remove name column from users table

[trca831]

[HER-16] Migration: Drop name column from User

Issue

https://raquelanaroman.atlassian.net/browse/HER-16

As a developer, I would like to remove tech debt from my app by removing the name column from users table.

Acceptance Criteria: create a migration that removes the name column from users

Links:

• https://raquelanaroman.atlassian.net/browse/HER-15
• https://raquelanaroman.atlassian.net/browse/HER-5
• https://raquelanaroman.atlassian.net/browse/HER-14

Changes

• Generated and ran a migration to Remove Name from Users table.
• Verified that change was completed by checking schema file.
• Updated Application code by removing (commenting out) name references in app/models/User.rb, spec/factories/Users.rb, and spec/models/User-spec.rb

Review Checklist

• I have documented my code with code comments.

[raquii]

Your migration is great! I want you to run this on your branch, though

git checkout HER-16-Migration-drop-name-column-from-Users
git fetch
git rebase origin/main
# resolve any conflicts, overwrite the credentials files entirely if you need to
# please DM me on slack if you want help resolving conflicts!
git push --force

[raquii]

This is good, actually. This is a good way to keep this branch scoped to what your changes should be.

However, we obviously will wait to deploy this until the blocking, dependent branches have shipped, so you will end up not needing this!

[trca831]

Hi @raquii Before I do this (what you commented above this regarding what to run on my branch), should I do a git pull?