{"push_id":559138176,"size":1,"distinct_size":1,"ref":"refs/heads/dylanrhodes/upstream_ora2_button","head":"7ca2d93949ee88f2dbaf7f9478d057c04a5585b6","before":"ae35d1d0905fbff238d70f5f7683496cef21c158","commits":[{"sha":"7ca2d93949ee88f2dbaf7f9478d057c04a5585b6","author":{"email":"d58ec75fb0d20bc5f4462cea0137c118c2f653f7@stanford.edu","name":"dylanrhodes"},"message":"Asynchronous download button for ORA2 data\n\nIncludes management command and instructor dashboard button which gather all open assessment response data for a given course in a user friendly format and output it as a .csv\n\nWIP finished collect_ora2_data management command, wired it to instructor dashboard UI, still needs to have logic factored out and exposed in instructor_analytics.basic\n\nRemove extraneous print statement\n\nRefactored sql querying logic into instructor.utils and wired the html button and management command to it there\n\nAdded comments to several methods\n\nAdded optional output_dir argument to collect_ora2_data management command and default behavior to write to stdout\n\nSimplified code structure in management command and comments per jrbl's PR comments\n\nResponded to jrbl's style comments, added test reference to check endpoint, and parameterized database name from django.conf.settings\n\nRemoved reference to django.conf.settings['DATABASE'] in favor of django's natural default\n\nRemoved extraneous import and whitespace per overlooked jrbl comment\n\nFactored db connection logic into common/djangoapps/util/query.py\n\nWhitespace changes\n\nThis implements a celery-ized button for downloading ORA2 responses via the instructor data download dashboard.\n\nWIP finished collect_ora2_data management command, wired it to instructor dashboard UI, still needs to have logic factored out and exposed in instructor_analytics.basic\n\nRemove extraneous print statement\n\nRefactored sql querying logic into instructor.utils and wired the html button and management command to it there\n\nAdded comments to several methods\n\nAdded optional output_dir argument to collect_ora2_data management command and default behavior to write to stdout\n\nSimplified code structure in management command and comments per jrbl's PR comments\n\nResponded to jrbl's style comments, added test reference to check endpoint, and parameterized database name from django.conf.settings\n\nRemoved reference to django.conf.settings['DATABASE'] in favor of django's natural default\n\nRemoved extraneous import and whitespace per overlooked jrbl comment\n\nFactored db connection logic into common/djangoapps/util/query.py\n\nWhitespace changes\n\nFirst stab at conversion to celery command from synchronous version\n\nWIP Converted ora2 report to asynchronous/celery format, still needs integration with progress table and proper S3 key.\n\nImplemented unit tests for instructor api, integrated ora2 celery job with pending tasks table, and cleaned up code for pylint\n\nRemoved extraneously included file from commits\n\nAdded more unit tests and reformatted whitespace\n\nFixed PEP8 violations, replaced pylint disable codes with string representations, and improved brittle unit test\n\nReformatted long string and removed extraneous redirect from javascript\n\nConflicts:\n\n\tcommon/djangoapps/util/query.py\n\tlms/djangoapps/instructor/tests/test_api.py\n\tlms/djangoapps/instructor/views/instructor_dashboard.py\n\tlms/djangoapps/instructor_task/api.py\n\tlms/djangoapps/instructor_task/tasks.py\n\tlms/djangoapps/instructor_task/tasks_helper.py\n\tlms/djangoapps/instructor_task/tests/test_api.py\n\tlms/djangoapps/instructor_task/tests/test_tasks_helper.py\n\tlms/envs/aws.py\n\tlms/envs/common.py\n\tlms/static/coffee/src/instructor_dashboard/data_download.coffee\n\tlms/templates/instructor/instructor_dashboard_2/data_download.html\n\nResolved additional merge conflicts\n\nCorrected PEP8 violations and removed extraneous imports to Stanford specific functions.\n\nWhitespace change and test case improvement.\n\nWhitespace change to fix PEP8 violation\n\nRemove string escape from management command data preprocesing\n\nResponded to comments on upstream PR\n\nSwitched status code and fixed Javascript problems and PEP8 violations\n\nAdd pylint disable comments\n\nSwitched unit tests to new status code for asynchronous report requests and fixed minor pep8 whitespace issues\n\nAdded missing import statement\n\nFixed pep8 violations\n\nFixed pep8 violation\n\nChanged whitespace\n\nFixing pep8 violation\n\nAttempt at pep8 fix\n\nDisabled pylint check for indent continuation in api_urls.py\n\nSmall change to event strings and whitespace issue\n\nAdded test cases for collect_ora2_data management command","distinct":true,"url":"https://api.github.com/repos/Stanford-Online/edx-platform/commits/7ca2d93949ee88f2dbaf7f9478d057c04a5585b6"}]}