Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
fayyazahmed66 opened this issue May 28, 2024 · 7 comments
Closed
Labels
redwood release testing Affects the upcoming release (attention needed)
Milestone

Comments

@fayyazahmed66
Copy link

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
Copy link
Author

label: redwood release testing

@mariajgrimaldi mariajgrimaldi added release testing Affects the upcoming release (attention needed) redwood and removed redwood release testing labels May 28, 2024
@DonatoBD
Copy link

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?

@Ian2012
Copy link

Ian2012 commented May 30, 2024

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

@crathbun428
Copy link

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).

@bmtcril
Copy link

bmtcril commented Jun 4, 2024

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.

@mariajgrimaldi mariajgrimaldi added this to the Redwood.3 milestone Sep 14, 2024
@mariajgrimaldi
Copy link
Member

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

@bmtcril
Copy link

bmtcril commented Sep 16, 2024

Yes, I believe that was fixed here: openedx/tutor-contrib-aspects#862

@bmtcril bmtcril closed this as completed Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
redwood release testing Affects the upcoming release (attention needed)
Projects
Development

Successfully merging a pull request may close this issue.

6 participants