ActoKids / AD440_W19_CloudPracticum

3 stars 1 forks source link

UI Forms: Delete Event>DELETE/events/{id} #50

Closed coultergeist closed 5 years ago

coultergeist commented 5 years ago

Connect forms to corresponding API endpoints: Delete Event>DELETE/events/{id}

Once assigned, annotate your estimated time. Then, keep regular updates on your issue. Comment your approach, tutorials, articles, updates, links, photos, etc. Comment often. When closing your issue include the following in your final comment.

Time estimated: Time spent: Pull Request link: Wiki link: Deployed Code link:

yahya09206 commented 5 years ago

Time estimated: 1-2 hours Time spent so far: 8 hrs Pull Request link: https://github.com/ActoKids/web-ui/pull/57 Wiki link: https://github.com/ActoKids/AD440_W19_CloudPracticum/wiki/UI-Forms:-Delete-Event-DELETE-events-%7Bid%7D Deployed Code link: deployed to prod but interfered with a bigger task like the filter function, so had to revert to not using delete. Tested by: Erik & Hau What I tested: I tested the delete function by checking off one event and deleting it, refreshing the page and making sure that event was actually deleted from the db and not showing up again on the dashboard.

Doing research on how to delete using react. First time working with this framework so the time estimated is based on learning some of the language as well to help me complete the task. Will update as I start to progress. I was few merges behind due to working with aws and travis ci when I started coding so trying to understand the codebase and learn how it was working took some time as well hence why my estimated time and time actually spent were way different.

-Shows event before deletion

Screen Shot 2019-03-20 at 4 15 50 PM

-Delete two events at the same time

Screen Shot 2019-03-20 at 4 16 12 PM

-Shows event that was under the previous two that were deleted from db

Screen Shot 2019-03-20 at 4 16 22 PM