Skip to content

add admin functions for wiping a version #3728

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

Closed
wants to merge 1 commit into from
Closed

add admin functions for wiping a version #3728

wants to merge 1 commit into from

Conversation

ajatprabha
Copy link
Contributor

Issue #3684: @agjohnson, I've added a wipe_build_environment method in the form class. But you mentioned about rethinking the process in order to have a reusable wipe view.
I'd like to discuss potential ways in which the functionality can be achieved.
Also, how about adding a proxy method to Version model to wipe a build environment? This will remove repeated code.

@RichardLitt RichardLitt requested a review from agjohnson March 5, 2018 20:47
@ericholscher ericholscher requested a review from a team June 7, 2018 14:52
@agjohnson
Copy link
Contributor

Unfortunately, the underlying issue was not for a site feature, but a django site admin feature. I'm going to close this issue as invalid, as we wouldn't use this feature as it stands. Feel free to repurpose your work as a site admin feature instead.

@agjohnson agjohnson closed this Jun 8, 2018
@ajatprabha ajatprabha deleted the issue_3684 branch June 9, 2018 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants