Skip to content
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

Improve latest download #288

Merged
merged 5 commits into from
May 12, 2022
Merged

Improve latest download #288

merged 5 commits into from
May 12, 2022

Conversation

JJ-8
Copy link
Contributor

@JJ-8 JJ-8 commented Nov 6, 2021

What is changed:

  • The platform in the URL is now detected by PlatformService.detect so you don't have to specify a perfect match anymore
  • The latest download now support arbitrary filenames/extensions and channels so you are not limited to a default file extension or channel
  • The releases/ directory is added to .gitignore

The documentation is updated accordingly.

JJ-8 added 5 commits November 6, 2021 11:21
The user can now specify custom file extensions to serve as latest release.
Now the user can get the latest version of a custom channel and custom flavor
Now the API is only accessible through authentication
Authentication is done through headers/cookies
@ArekSredzki ArekSredzki merged commit 2aacccf into ArekSredzki:master May 12, 2022
@JJ-8
Copy link
Contributor Author

JJ-8 commented May 12, 2022

Sorry @ArekSredzki I was not aware that this PR was merging my master into your master. After creating this PR, I have pushed 8d2099f which breaks the latest download page (on purpose). This is not something you want to have in your master. I propose to revert this PR and that you cherry pick ever commit except 8d2099f

ArekSredzki added a commit that referenced this pull request May 12, 2022
ArekSredzki added a commit that referenced this pull request May 12, 2022
@ArekSredzki
Copy link
Owner

Thanks for the heads up @JJ-8. My bad for not having tested it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants