Skip to content

Will Baxter#134

Open
MrStashy wants to merge 25 commits intoboolean-uk:mainfrom
MrStashy:main
Open

Will Baxter#134
MrStashy wants to merge 25 commits intoboolean-uk:mainfrom
MrStashy:main

Conversation

@MrStashy
Copy link

@MrStashy MrStashy commented Jul 1, 2024

No description provided.


async function updateCustomerById(req, res) {
const id = Number(req.params.id);
if (isNaN(id)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Top marks

Comment on lines +24 to +30
screening: {
id: ticket.screening.id,
movieId: ticket.screening.movieId,
startsAt: ticket.screening.startsAt,
createdAt: ticket.screening.createdAt,
updatedAt: ticket.screening.updatedAt,
},
Copy link
Contributor

Choose a reason for hiding this comment

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

screening: { ...ticket.screening }


let ticket;
try {
ticket = await prisma.ticket.create(request);
Copy link
Contributor

Choose a reason for hiding this comment

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

return await prisma.ticket.create(request);

Copy link
Contributor

Choose a reason for hiding this comment

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

return prisma.ticket.create(request) await is redundant if you are immediately returning

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants