Issue #20 seemed to ask only for creating the column in the database. I'm not super familiar with Sequelize but I tried adding a delete column to the event schema. I think what's required after this is:
updating GET api/v1/event/past and GET api/v1/event/future and GET api/v1/event/:uuid? to only return events where deleted = false
updating DELETE api/v1/event/:uuid? to set deleted = true
don't change PATCH api/v1/event/:uuid?, but still allow people to manually use it to toggle deleted in case they want to undelete events
migrate existing events to the new schema? I read a little bit about how to do that, I'm not familiar with how the production database works so I assume someone else could take care of this.
but I'm not sure, so please clarify if I'm wrong. I'm not very familiar with the codebase so it's possible I edited the completely wrong thing.
howardwang15
commented
5 years ago
Issue #20 seemed to ask only for creating the column in the database. I'm not super familiar with Sequelize but I tried adding a delete column to the event schema. I think what's required after this is:
but I'm not sure, so please clarify if I'm wrong. I'm not very familiar with the codebase so it's possible I edited the completely wrong thing.