openedx-unsupported / edx-analytics-pipeline

GNU Affero General Public License v3.0
91 stars 116 forks source link

For module engagement, parse the course ID from the URL #849

Closed doctoryes closed 3 years ago

doctoryes commented 3 years ago

... when not present in the context.

https://openedx.atlassian.net/browse/DENG-606

Analytics Pipeline Pull Request

Make sure that the following steps are done before merging:

codecov[bot] commented 3 years ago

Codecov Report

Merging #849 (95a22f3) into master (6f10185) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #849   +/-   ##
=======================================
  Coverage   74.02%   74.02%           
=======================================
  Files         208      208           
  Lines       23887    23887           
=======================================
  Hits        17683    17683           
  Misses       6204     6204           
Impacted Files Coverage Δ
edx/analytics/tasks/insights/module_engagement.py 79.13% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 6f10185...95a22f3. Read the comment docs.