-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Edit ticket page #32
Edit ticket page #32
Conversation
@Val-Bustamante this branch breaks the seed file: |
@Val-Bustamante |
@Val-Bustamante |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look good!
Github says the PR can be merged without issue, but doing a rebase with main seems to have a lot of conflicts. @Val-Bustamante @bigguyhere @dbajollari1 lets try to review the conflicts together today before I break something trying to do it myself. |
Found another issue on the PR. When editing a ticket and adding an owner, it does not update that owner’s list of owned tickets This also applies to unassigning an owner not removing the ticket from their list. Not sure if this is for @Val-Bustamante for the page/routes/changes to the data function or for @dbajollari1 for the data functions originally. |
… will cause error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed issues with last review myself
Ignore this is propper behavior |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Think we're good now!
Fixed edit ticket page so that the fields are automatically filled with the values of the original ticket. Added way for users to add owners. Made deadline be of type date time when inserted into mongo.