David Ormsbee
David Ormsbee
@KristinAoki: This is fantastic! Thank you!
I am not really competent at writing MySQL stored procedures, but this seems right to me. Certainly the logical mappings in the description and the options you give in the...
@Faraz32123, @regisb: What do you need at this point in order to get this PR merged?
@hurtstotouchfire: Wanted to make sure you folks were aware of this PR. This introduces the switching code between old and new forums. It should not affect the default site behavior,...
Ah, I see, thank you.
Apologies for the delay in merging this. 2U had requested time to get their stage environment ready for testing, and they're also rolling out a database config change to prod...
2U is all clear now, so this can be merged as soon as conflicts are resolved. Thank you for your patience.
@Faraz32123, @regisb: Just want to flag that there's a test failure.
I think that it'll be important for LearningPackages to have independent lifecycles. Foreign keys across LearningPackage boundaries are fine, but they should have `on_delete=models.SET_NULL`, and behave properly when the originating...
> What do you think (overall) of the current implementation? Overall, I think the approach is good–especially given where we are today. My only review comments are on lower-level details.