-
Notifications
You must be signed in to change notification settings - Fork 11
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
Latest prerelease does not fetch properly #12
Comments
It seems to skip all versions that start with a number. |
Number comes before alphabet (at least in computer) and it sort in descending order. So that's why. |
well it's not sorted alphabetically, it's sorted by the release time. In this list at least. |
I'm talking about this list which sort alphabetically and we only get top 50 from them. So that why it miss some recent releases. |
@beam41 do you know if a GitHub app (no specific repo installation) could be used to authenticate either the REST API or the GraphQL API? |
GitHub Apps can be use to authenticate both api. |
Actually I think you mean "OAuth Apps" |
@beam41 no, I mean a GitHub App, one that you create on GitHub with a private key and authenticate using a JWT. So the user of the launcher does not need to authenticate, it just uses a shared private/public key for the GitHub App. I am just wondering whether the GitHub App can use the GraphQL API for finding releases without needing to be enabled/installed for a specific repo - since it is a public repo. |
https://developer.github.com/changes/2018-04-30-graphql-supports-github-apps/ |
@beam41 that is a shame. I am not sure how best to resolve this issue without requiring the user to authenticate with a GitHub account. There are over 3,300 releases on the repo and the maximum number of releases you can query is 100, so that is at least 33 / 60 requests to get them all and it takes time to do a query that large. |
The best I can think is small api server fetch Github releases from Graphql |
As a temporary measure, |
Currently launcher show latest prerelease at cf6de24 even though we have newer commit on develop.
I figure out what happen. If we use postman to fetch release from github api directly.
First tag name will be match with first result in the release page and so on.
That mean this api sort by tag name string same as the release page, and that why newer commit on develop doesn't appear because tag start with f come first.
Look at the documentation, there is no way we can sort by publish date, but we might be able to fix by expand result to cover all commit in major version. (Which is not really viable)
Graphql version can be sort but need authentication though.
The text was updated successfully, but these errors were encountered: