-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
Fix inconsistency of the data returned by the API #747
Fix inconsistency of the data returned by the API #747
Conversation
Note that this PR will not fix the difference of counts returned by |
code change looks OK but @johnoliver to check on the semantics here |
@johnoliver Note that I have found that in the following method, data are NOT filtered to compute the Total. In GitHubDownloadStatsCalculator, line 55
|
We chose to add as a parameter the list of types we want to include To fix the related issue (adoptium/dash.adoptium.net#385), we have to modify calls from the dashboard to retrieve all releaseTypes to compute drilldowns. In api.tsx: |
This PR is to resolve (partially) this issue adoptium/dash.adoptium.net#385
There is some inconsistency in data returned by the API.
Two endpoints do not return same data because they do not filter values in the same way.
Filter is
.filter { it.release_type == ReleaseType.ga }
line 67 getTotalDownloadStats()
⚠️ do not have the filter
and
line 93 getTotalDownloadStatsForTag()
✔️ have a filter
➡️ I have added the filter to both function. Maybe I should remove the filter from the second, let me know.
Checklist
mvn clean install
build and test completes