feat: statistics API #4
Conversation
shadinaif
force-pushed
the
shadinaif/dashbord-stats-learners-count
branch
2 times, most recently
from
3d9a001 to
375adc9
Compare
shadinaif
force-pushed
the
shadinaif/dashbord-stats-learners-count
branch
6 times, most recently
from
bc6b3af to
514889a
Compare
shadinaif
force-pushed
the
shadinaif/dashbord-stats-learners-count
branch
2 times, most recently
from
22d6afe to
193dd9c
Compare
shadinaif
force-pushed
the
shadinaif/dashbord-stats-learners-count
branch
from
193dd9c to
7daf9dc
Compare
There was a problem hiding this comment.
@shadinaif please go ahead and merge this pull request. I wasn't aware it was blocked by review.
| ) | ||
| ) | ||
| ).filter( | ||
| Q(courses_count__gt=0) | Q(has_site_login=True) |
There was a problem hiding this comment.
We need the list of users related to the tenant. So, the user must be enrolled in one of the tenant's courses, or have a site login in the tenant's site
| from futurex_openedx_extensions.helpers.tenants import get_course_org_filter_list, get_tenant_site | ||
|
|
||
|
|
||
| def get_learners_queryset(tenant_ids: List, search_text: str = None) -> QuerySet: |
There was a problem hiding this comment.
This query is a bit slow on production. Any idea how can we make it faster?
There was a problem hiding this comment.
I believe the issue is not this particular query. It's the other helper queries, you'll find a few methods with TODO to add cache. These will increase the performance significantly as far as I can see
There was a problem hiding this comment.
if caching didn't fix it, we might need to do something with UserSignupSource. We need it to record logins, not just signups. So we can use it to directly filter the user by site.
Currently, there are too many records for users with enrollments in a site, but without a record in UserSignupSource
feat: statistics API