Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete csv files #12

Open
schorsch opened this issue Jan 24, 2012 · 2 comments
Open

Delete csv files #12

schorsch opened this issue Jan 24, 2012 · 2 comments
Labels

Comments

@schorsch
Copy link
Member

Allo the user to delete uploaded files, need to check:

  • what to do with the mappings
  • also possible old imports ?
@vijendra
Copy link
Contributor

In two ways we can handle this.

  1. Hard delete. We can show warning message about the existing mappings and imports. And if the user wants to proceed we can delete.
  2. Soft delete. We can flag it as deleted and show only active attachments.

What do you suggest? Also right now attachment_id field is not present in mappings table. Hence attachment_id is not stored.

@schorsch
Copy link
Member Author

We should go for hard deletion

  • warn & delete related imports
  • silently ignore missing files(they dont persist on heroku)
  • keep mappings .. to be reused

vijendra added a commit to vijendra/CSV-Import that referenced this issue Feb 4, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants