Update Table: User | Model | API | Test cases - #616
Conversation
cc66bb1 to
c93f0e2
Compare
|
I rebased to main and updated the migration file numbering, making sure tests and linters are passing. I haven't looked at the code yet. |
fyliu
left a comment
There was a problem hiding this comment.
Thanks for working on this! I think we need some clarification on the requirements for this to be sure. I will bring it up at the meeting.
e3c7118 to
d55d2e9
Compare
d55d2e9 to
58c7626
Compare
|
egcuriel
left a comment
There was a problem hiding this comment.
Everything looks good, the changes reflect the updated requirements. Should these changes be incorporated into PRs waiting to be merged/Issues currently open, or should this PR include any changes those PRs have incorporated/Issues currently open?
|
@egcuriel I think it's fine if PRs end up conflicting with one another. If the PR was worked on for a long time, it should be rebased to the latest |
Fixes #601
What changes did you make?
Why did you make the changes (we will use this info to test)?
Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)