-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: create queries for the 4th and 5th card with more context if data is partial #67
Comments
resolved in #76 |
This has been partially resolved by #76, we need the following 5th statistic:
We've shared this stat with the data team and it seems to be approved. |
nice! thank you @OmarIthawi ! |
@shadinaif, please let me know when this is implemented in staging so we can add it the frontend. |
@tehreem-sadat Please let me know if you or Shadi can get to this task any time soon. It's about time that we add this metric to our homepage so it looks complete :) cc: @shadinaif |
@OmarIthawi started working on it yesterday, I;ll let you know when it is done. |
@OmarIthawi as I remember from a previous discussion; the number of passing students is counted according to certificates. Please let us know if that changed for any reason |
@shadinaif I'm unsure what you mean. All the existing cards are great!! We need the fifth one as mentioned here #67 (comment)
I hope that makes it clearer |
@OmarIthawi I mean: your proposed equation contains |
Yes, it's Thanks for the calrification @shadinaif. |
Same issue:
The text was updated successfully, but these errors were encountered: