-
Notifications
You must be signed in to change notification settings - Fork 217
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
hub: cleanup data of deleted projects #7667
base: master
Are you sure you want to change the base?
Conversation
huge +1! It'll be a good foundation for doing this better in general. |
4c6270e
to
5d57624
Compare
5d57624
to
af28b7f
Compare
0646bf4
to
37d39b0
Compare
37d39b0
to
59131f7
Compare
… database package
35c17dd
to
5d7c4aa
Compare
c725c35
to
ff5f370
Compare
ff5f370
to
d8f481c
Compare
31a9137
to
39d0065
Compare
735e039
to
f4cabc5
Compare
7de0696
to
e83d148
Compare
e83d148
to
55d2a11
Compare
"file_access_log", | ||
"jupyter_api_log", | ||
"openai_chatgpt_log", | ||
"blobs", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Think twice about blobs. I would not delete the blobs.
I'm working on implementing an exam mode for courses. This must involve creating a new project per student for each exam, because it seems it is VERY important to instructors that students don't have access to the exam when the exam is over (and also temporarily don't have access to their own work). This means that the student should be removed from the exam project and their work collected when the exam ends. It would be natural at that point to delete the exam project. I understand a big point of this PR is to make it so when you delete a project, all TimeTravel edit history also gets permanently purged. I'm just trying to figure out what to do here... No comment is needed, probably -- I just wanted to write this down. |
b770b46
to
5d797ef
Compare
Description
TODO
EXTRA
"modernize" visibility and delete controls (popconfirm, switch) … text and so on are the same.
Checklist: