-
Notifications
You must be signed in to change notification settings - Fork 10
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
Updated query according to model update #41
Conversation
28d090e
to
69ff5b3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for getting this ready to go
I expect this to fix each of:
program_course_enrollment = enrollment.programcourseenrollment_set.all() | ||
if program_course_enrollment.exists(): | ||
program_course_enrollment = program_course_enrollment.first().program_enrollment | ||
enrollment_dict['student_uid'] = program_course_enrollment.external_user_key | ||
else: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will the program_enrollment
object be the same for all the PCE? Just confirming this to avoid any incomplete/incorrect data export.
@AhtishamShahid Please rebase your branch as there has been new changes in master. |
69ff5b3
to
c8c8222
Compare
updated mocker and data Refector added pip version for tox test added pip version for tox test added pip version for tox test added pip version for tox test updated version updated get to first updated get to first updated version
c8c8222
to
969b8f7
Compare
JIRA: PROD-1352
Issue :
unable to download staff graded reports because there was recent change https://github.com/edx/edx-platform/blame/7f53a9ab9b4abfc550d3dc16e54cf63502330346/lms/djangoapps/program_enrollments/models.py#L110 so updated query and test accordingly to fix this issue.
Merge checklist:
Post merge:
finished.