bszabo
bszabo
@iamCristYe I work in the same group as Connor Haugh and have started looking at this PR, especially with regards to unit tests. I'm well along with a unit test...
@iamCristYe I tried pushing my unit tests onto a new branch on this PR's repo, but don't appear to have appropriate permissions. Can you help with this? Do you have...
I just created PR https://github.com/iamCristYe/edx-platform/pull/1 with my proposed unit tests. @iamCristYe is specified as a reviewer, but I was unable to add @mariajgrimaldi as one. The PR is for a...
@iamCristYe @mariajgrimaldi Where do we stand with regards to this PR? I've run a branch with these changes in it against devstack and confirmed that for a small set of...
Hi @mariajgrimaldi. I, too, am trying to understand what work remains. From your perspective, other than having the new comments you posted addressed, and having the unit tests merged, is...
In side channel slack conversation, Alexander confirmed that none of these changes broke the edx-enterprise build, and that the extraction of the course enrollment from the student model looks largely...