Skip to content
This repository has been archived by the owner on Oct 16, 2024. It is now read-only.

Ticket iteration #21

Merged
merged 6 commits into from
Nov 11, 2023
Merged

Ticket iteration #21

merged 6 commits into from
Nov 11, 2023

Conversation

rsotoacu
Copy link
Collaborator

This branch has the same things from ticket branch but some things have been added regarding ticket iteration.

…ed to the shcema (ticket.js) and CRUD operations now alter the iterations as well.
@gtaEPIC gtaEPIC added this to the Backend milestone Nov 10, 2023
@gtaEPIC gtaEPIC added the REQUIRED Required for submission label Nov 10, 2023
Copy link
Owner

@gtaEPIC gtaEPIC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Iteration should be an array. This is what will log the different actions done.

models/ticket.js Outdated Show resolved Hide resolved
controllers/ticketController.js Outdated Show resolved Hide resolved
controllers/ticketController.js Outdated Show resolved Hide resolved
controllers/ticketController.js Outdated Show resolved Hide resolved
@gtaEPIC gtaEPIC linked an issue Nov 11, 2023 that may be closed by this pull request
6 tasks
Copy link
Collaborator

@imirza7 imirza7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!!

Made iteration an array and added newStatus
With iteration being an array, creating a ticket creates an array with one iteration
Made editing a ticket add a new iteration to the array
Made disabling a ticket add an iteration to say it was disabled
Copy link
Owner

@gtaEPIC gtaEPIC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good now

@gtaEPIC gtaEPIC merged commit 8a2011d into main Nov 11, 2023
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
REQUIRED Required for submission
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Ticket iterations
5 participants