Skip to content

Conversation

@holytoastr
Copy link
Collaborator

@holytoastr holytoastr commented Jul 2, 2016

Adds soft delete functionality on the backend with the paranoia gem. Includes tests (which pass! yay!).

Will add delete option to admin dashboard in next PR.

@jkeam The images table that you created disappears when I run db:migrate. I don't know why.

it "can be REALLY destroyed" do
a_user.really_destroy!
expect(a_user.destroyed?).to eq(true)

Choose a reason for hiding this comment

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

Extra empty line detected at block body end.

@jkeam
Copy link
Collaborator

jkeam commented Jul 4, 2016

Which images table do you mean? Which route do you mean (whats the url, the localhost development one is fine)? I'll double check it and fix it if I borked something :)

@holytoastr
Copy link
Collaborator Author

@jkeam I think I fixed the issue. Had to do some git magic. Not really sure what happened, but the schema includes your changes now.

@jkeam
Copy link
Collaborator

jkeam commented Jul 11, 2016

I think this looks good 👍 . If there are no objections from the team I'm ok with merging this in. Great job!

@jkeam
Copy link
Collaborator

jkeam commented Jul 11, 2016

Just adding in the issue that I created after the fact to track this functionality.

#73

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants