Skip to content

Test + fix for crash when block is deleted after students have used it - #25

Merged
bradenmacdonald merged 1 commit into
masterfrom
fix-author-changes
Apr 30, 2015
Merged

Test + fix for crash when block is deleted after students have used it#25
bradenmacdonald merged 1 commit into
masterfrom
fix-author-changes

Conversation

@bradenmacdonald

Copy link
Copy Markdown
Member

@Kelketek

Copy link
Copy Markdown
Member

@bradenmacdonald Please ping me once you've merged the other task and rebased this atop that so that the tests pass.

@bradenmacdonald

Copy link
Copy Markdown
Member Author

@Kelketek Tests passing now :)

@smarnach

Copy link
Copy Markdown
Contributor

👍 Code looks good. While testing this, I managed to mess up my solutions devstack again, and I don't have time this evening to fix it. Since this PR needs a only a single reviewer anyway, I'll leave the testing to Jonathan.

@Kelketek

Copy link
Copy Markdown
Member

👍 Works for me! Looks good, too.

bradenmacdonald added a commit that referenced this pull request Apr 30, 2015
Test + fix for crash when block is deleted after students have used it
@bradenmacdonald
bradenmacdonald merged commit d9f2097 into master Apr 30, 2015
@bradenmacdonald
bradenmacdonald deleted the fix-author-changes branch April 30, 2015 23:57
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