feat: update UserEmploymentHistory to use ModernJobTitle FK#656
feat: update UserEmploymentHistory to use ModernJobTitle FK#656mrigasiyer wants to merge 2 commits into
Conversation
Replaces the title CharField with a modern_job_title ForeignKey referencing the ModernJobTitle table. Updates admin, serializer, and tests accordingly. Closes hackforla#618
c384bda to
e66d036
Compare
|
I had to update the main branch to fix some formatting, so I rebased this branch to that. There's no code changes. |
|
The changes look great and works well! I just wanted to discuss another change related to this table at the meeting before approving. |
|
hi fang! I'm actually going to be out traveling for the next week and half so I won't be able to join a meeting for while. I'm open to discuss on slack. PM me at your best convenience! |
|
@mrigasiyer We'll see you when you get back In the meantime, we updated the issue to remove the |
fyliu
left a comment
There was a problem hiding this comment.
The code reads fine. Builds, runs, tests pass. Thanks!
We did have to make a change to the issue that needs a field removed. Please see the issue description. Thanks!
Fixes #618
Replaces the title CharField with a modern_job_title ForeignKey referencing the ModernJobTitle table. Updates admin, serializer, and tests accordingly.
What changes did you make?
Why did you make the changes (we will use this info to test)?