Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

remove OEP-10 so this repo is no longer shipped with openedx releases. #841

Closed
wants to merge 1 commit into from

Conversation

macdiesel
Copy link
Contributor

Analytics Pipeline Pull Request

Make sure that the following steps are done before merging:

  • If you have a migration please contact data engineering team before merging.
  • Before merging run full acceptance tests suite and provide URL for the acceptance tests run.
  • A member of data engineering team has approved the pull request.

@macdiesel macdiesel requested review from natabene and estute November 3, 2020 19:20
@natabene
Copy link

natabene commented Nov 3, 2020

@macdiesel I am for it, but cannot really review from engineering perspective. @nedbat , any comments?

@codecov
Copy link

codecov bot commented Nov 3, 2020

Codecov Report

Merging #841 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #841   +/-   ##
=======================================
  Coverage   74.01%   74.01%           
=======================================
  Files         208      208           
  Lines       23861    23861           
=======================================
  Hits        17661    17661           
  Misses       6200     6200           

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 a58bd93...8a52a86. Read the comment docs.

@kdmccormick
Copy link

kdmccormick commented Dec 13, 2023

Relevant DEPR: openedx/public-engineering#36

EDIT: That DEPR is just for one view, not the whole system.

@kdmccormick kdmccormick force-pushed the bbeggsretire-from-openedx-release branch from 8a52a86 to 68be3de Compare December 13, 2023 17:27
Copy link

@kdmccormick kdmccormick left a comment

Choose a reason for hiding this comment

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

Wait, I'm not sure Insights deprecated. Just the learner view and supporting code is.

Is there another DEPR ticket I'm missing?

@nedbat nedbat closed this Dec 13, 2023
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.

5 participants