Open Caleb-Cohen opened 1 year ago
@Caleb-Cohen I'd like to take this issue if I could get assigned whenever you have a chance.
A couple things I noticed from my first look at this issue
approveEvents
method should take a single event or many to be approved all at once. I think the second case makes the most sense since you could also just provide a single event as well, but it might also make sense to add separate endpoints for approveEvent
(singular) and approveEvents
(multiple) for clarity. I just wanted to get clarification before making a decision.PATCH
would be more appropriate here instead of POST
since we should only be updating a single property on each event.
Related Issues Project: Admin Dashboard Depends on:
408
Description Create a method in the
events.js
controller calledapproveEvents
Method will change specific event objects status to approved Create post route inevents.js
Create axios requestAcceptance Criteria Test updated event object has been changed from pending to approved