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.

Sign up for GitHub

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

Jump to bottom

Need to delete old exports #1056

Closed
taylortom opened this issue Feb 22, 2016 · 2 comments
Closed

Need to delete old exports #1056

taylortom opened this issue Feb 22, 2016 · 2 comments
Assignees
@taylortom
Labels
D: easy Straightforward issues, small time commitment S: merged Completed, reviewed, and merged issues T: enhancement Adding additional functionality
Milestone
0.5.0 Import/Export

Comments

@taylortom
Copy link
Member

At the moment, these are all stored indefinitely.

Simple solution is to only keep one export per course. Any other ideas?
@taylortom taylortom added T: enhancement Adding additional functionality D: easy Straightforward issues, small time commitment labels Feb 22, 2016
@taylortom taylortom added this to the 0.1.6 milestone Mar 1, 2016
@taylortom
Copy link
Member Author

Better solution could be to store one per-user/per-course, then delete on logout

@taylortom taylortom self-assigned this Mar 3, 2016
@taylortom
Copy link
Member Author

Have altered exports to only allow a single zip per-user (with the zip name now being the user's ID).
Couldn't see a way to tap into a logout event from the backend, so haven't implemented that. Is this a suitable solution? @brian-learningpool @dancgray

@taylortom taylortom mentioned this issue Mar 3, 2016
Merged
@taylortom taylortom added the S: awaiting-review Completed issues waiting on reviews label Feb 26, 2018
@dancgray dancgray added the S: merged Completed, reviewed, and merged issues label Mar 26, 2018
@taylortom taylortom removed the S: awaiting-review Completed issues waiting on reviews label Apr 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@taylortom taylortom

Labels
D: easy Straightforward issues, small time commitment S: merged Completed, reviewed, and merged issues T: enhancement Adding additional functionality
Projects
None yet
Milestone
0.5.0 Import/Export
Development

No branches or pull requests
2 participants
@taylortom @dancgray