Skip to content
This repository was archived by the owner on Nov 15, 2024. It is now read-only.

Staging - #446

Merged
muhammadnadeem merged 34 commits into
masterfrom
staging
May 3, 2018
Merged

Staging#446
muhammadnadeem merged 34 commits into
masterfrom
staging

Conversation

@muhammadnadeem

Copy link
Copy Markdown

No description provided.

Muhammad Osama Arshad and others added 30 commits March 28, 2018 21:30
remove all reference to Crimea, Cuba, Iran and North Korea
add alert message when course has ended and move account activation c…
* Improve: Add target course model

* Improve: add course analytics management command

* Improve: basic version of data email complete

* Improve: add course performance data

* Improve: add submission data and anonymous user ids

* Improve: add informative data

* Improve: add ora file url

* Improve: add option for emails in model

* Improve: remove unnecessary data variable

* FIX: correct user data heirarchy

* REFACTOR: change model name and models

* IMPROVE: split the get_course_analytics command code into helper functions

* IMPROVE: optimize anonymous user id requests, fix relative imports

* IMPROVE: add newline at the end of urls.py
* Improve: Add target course model

* Improve: add course analytics management command

* Improve: basic version of data email complete

* Improve: add course performance data

* Improve: add submission data and anonymous user ids

* Improve: add informative data

* Improve: add ora file url

* Improve: add option for emails in model

* Improve: remove unnecessary data variable

* FIX: correct user data heirarchy

* REFACTOR: change model name and models

* IMPROVE: split the get_course_analytics command code into helper functions

* IMPROVE: optimize anonymous user id requests, fix relative imports

* IMPROVE: add newline at the end of urls.py

* FIX: use correct retrieval method from answer dict

* IMRPOVE: use better syntax in get_file_url function

@zee-pk zee-pk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@muhammadnadeem
muhammadnadeem merged commit bb59432 into master May 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants