nelc / futurex-openedx-extensions

Open edX APIs and reporting tools
Other
0 stars 0 forks source link

bug: SREI production instance shows all courses count even if they're for testing purposes #7

Open OmarIthawi opened 2 months ago

OmarIthawi commented 2 months ago

Go to https://dashboard.futurex.sa/ and check SREI course results

Actual result: 17 Expected result: 3 (or anything other than 17)

image

shadinaif commented 2 months ago

thank you @OmarIthawi . For now, I'll exclude all courses with catalog_visibility not in ('about', 'both'). That will make the value 4 (three active and one archived). Do you think we need a global filter in the dashboard to include/exclude hidden courses? like a checkbox somewhere on the page (not a settings,,, settings will be confusing and hard to find)

OmarIthawi commented 2 months ago

thank you @OmarIthawi . For now, I'll exclude all courses with catalog_visibility not in ('about', 'both'). That will make the value 4 (three active and one ended). Do you think we need a global filter in the dashboard to include/exclude hidden courses? like a checkbox somewhere on the page (not a settings,,, settings will be confusing and hard to find)

I think we should have as few settings as possible.

Your suggestion works regarding catelog_visibility.

OmarIthawi commented 1 month ago

@shadinaif kindly keep this issue open until it's closed in production. Last time I checked it's still an issue for SREI in https://dashboard.futurex.sa/ .