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: Pre-release patches toggle #1785
base: dev
Are you sure you want to change the base?
Conversation
Why limit it to alternative sources? You can pass ?dev=true to achieve the same with the ReVanced API |
Are changelogs/updates considered as well? @Ushie I am not sure but are queries ignored by cache on CloudFlare? Because then, using the dev query has no effect. |
I already tried ?dev=true at ReVanced API, but it didn't effect. If the cache issue will be fixed, I think I should wait for it.
Yes. These lines change the data source of patches update dialog according to the settings. revanced-manager/lib/ui/views/home/home_viewmodel.dart Lines 482 to 490 in e88e997
|
One thing I noticed after building this branch (rebased) was that checking for |
Can you share the apk here please |
@nivranaitsirhc sure [redacted] |
It works like a charm! Thanks! |
Add a toggle for pre-release patches to the alternative source settings.
Closes: #821
This PR also fixes an problem where pre-release Integrations are being used when using alternate sources.
At this time, Patches are referring
GitHubAPI#getLatestPatchesRelease()
that doesn't include pre-releases, but Integrations are referringGitHubAPI#getLatestRelease()
that includes pre-releases.This problem actually caused patched apps to crash recently in crimera/piko patches (an alternative source) when they released a pre-releases.
(Actually, this is the main motivation)
getLatestReleaseFile()
ingithub_api.dart
was removed because it's unused