wg-build-test-release icon indicating copy to clipboard operation
wg-build-test-release copied to clipboard

[Test failure] <TC_AUTHOR_REDWOOD_7>: < un-enrollment is not reflecting>

Open fayyazahmed66 opened this issue 1 year ago • 5 comments

Release

Redwood

Expected behavior

un-enrollment should also reflected in this stat.

Actual behavior

un-enrollment isn't reflected in this stat.

Steps to reproduce

  1. Go to instructor tab > Reports
  2. Click on the course dashboard stat
  3. unenroll from this course with a learner user
  4. check stat again in current enrolees
Screenshot 2024-05-28 at 2 43 40 PM

Additional information

https://redwood.demo.edly.io/courses/course-v1:OpenedX+DemoX+DemoCourse/instructor#view-aspects

fayyazahmed66 avatar May 28 '24 09:05 fayyazahmed66

label: redwood release testing

fayyazahmed66 avatar May 28 '24 09:05 fayyazahmed66

Hi @fayyazahmed66 thanks for the report ! I'm a bit lost on aspects but I have the perfect person who can help us understand what's going on 😊

@Ian2012 can you give us a hand here please?

DonatoBD avatar May 30 '24 15:05 DonatoBD

Thanks for the report, the issue is that the query counts enrollment_events instead of only counting the latest enrollment event per learner.

Ian2012 avatar May 30 '24 15:05 Ian2012

Not a release blocker, but should be fixed since current enrollees should show the number of currently enrolled learners, not total enrollees (all learners who have enrolled in the course).

crathbun428 avatar Jun 03 '24 19:06 crathbun428

If this isn't fixed in Daniela's next round of changes we'll fix it as a fast follow, it should be a quick fix I just don't want it to get lost integrating the next set of dashboard changes.

bmtcril avatar Jun 04 '24 19:06 bmtcril

Hi there, @Ian2012 @bmtcril. Could you let me know if this was ever fixed? Thanks.

mariajgrimaldi avatar Sep 16 '24 11:09 mariajgrimaldi

Yes, I believe that was fixed here: https://github.com/openedx/tutor-contrib-aspects/pull/862/

bmtcril avatar Sep 16 '24 13:09 bmtcril