Skip to content

Delete crate script should delete categories and badges too #698

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

Merged

Conversation

carols10cents
Copy link
Member

When deleting the nul crate, I couldn't use the delete crate script for a number of reasons, one being that there were categories and when I added categories I forgot to add deleting them to this script :(

@sgrif
Copy link
Contributor

sgrif commented May 1, 2017

What do you think about cascading the deletion at the database level?

@carols10cents
Copy link
Member Author

carols10cents commented May 1, 2017

I'd be into that. Would make this script a lot simpler! Will modify this PR.

@alexcrichton alexcrichton merged commit cb04fe4 into rust-lang:master May 2, 2017
@alexcrichton
Copy link
Member

Oh er a db removal sounds fine, I'll also merge that sort of PR!

@carols10cents carols10cents deleted the delete-categories-and-badges branch May 2, 2017 15:57
@carols10cents
Copy link
Member Author

Meh, i'll make an issue to remind me :)

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