-
Notifications
You must be signed in to change notification settings - Fork 165
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
update OSO endpoint and queries #3672
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
known issue: when browsing through applications in explorer, Impact Stats will not populate the page after visiting a project with no stats (even if the application you're looking at does have them). page refresh fixes this. caching/cookie issue? |
believe this is fixed. improved logic determining when stats are queried to account for circumstances where stats have already been queried, but not for the currently viewed project. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one style suggestion
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
Fixes: #3669
Description
https://linear.app/gitcoin/issue/PAR-438/update-oso-endpoint-and-queries
With current OSO models it is not straightforward to retrieve code velocity data. Updated that field to instead reflect OSO's
activeDeveloperCount6Months
field for a sense of active developers.Checklist
This PR:
@ts-ignore
, unjustified optional values).