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

WIP: Check for updates #3855

Closed
wants to merge 9 commits into from
Closed

WIP: Check for updates #3855

wants to merge 9 commits into from

Conversation

nikclayton
Copy link
Contributor

This is v22 with some code to check for updates. More details when I've done more testing. In the meantime, I need an APK from CI so it's been signed with the right keys.

Nik Clayton added 7 commits July 14, 2023 20:55
Cherry pick of 2537617
`OldTargetApi` default behaviour is to warn (https://googlesamples.github.io/android-custom-lint-rules/checks/OldTargetApi.md.html)

Set it back to that, so that CI runs on runners with newer versions of the SDK installed do not fail.
…or-updates

# Conflicts:
#	app/src/main/java/com/keylesspalace/tusky/settings/SettingsConstants.kt
#	app/src/main/res/values/strings.xml
@connyduck
Copy link
Collaborator

I'm against deploying different flavors to F-Droid and Google Play. Unnecessary additional complexity. Also, including a Google Play library will give us a "Upstream non free" antifeature on F-Droid which is a no-go imo.

@nikclayton
Copy link
Contributor Author

I'm against deploying different flavors to F-Droid and Google Play. Unnecessary additional complexity.

Some more complexity, yes, but we don't know it's unnecessary yet. At the time of writing approx. 30K users (about 18%) are still on a version < 22 (which was released over a month ago

Also, including a Google Play library will give us a "Upstream non free" antifeature on F-Droid which is a no-go imo.

Doesn't have to. At the moment the release process attaches APKs from Google Play to the GitHub release, but we could change that, and attach an APK built from the fdroid flavor instead.

Then "Upstream non-free" wouldn't apply.

@charlag
Copy link
Collaborator

charlag commented Jul 15, 2023

I would also not like different flavors again unless we really really need jt

@Tak
Copy link
Collaborator

Tak commented Jul 15, 2023

I thought it was an explicit project goal not to rely on google services and/or non-free dependencies?

@connyduck
Copy link
Collaborator

Yes @nikclayton please don't waste you time on this.

(Theoretically we could implement a custom update check, e.g. by hosting a version.json on tusky.app and periodically querying that, but that will add a new step to releasing and its probably not as easy as it sounds...)

@nikclayton
Copy link
Contributor Author

#3882 is a much cleaner version of this which works for F-Droid and GitHub releases too, and doesn't introduce "Upstream non free" on F-Droid.

@nikclayton nikclayton closed this Jul 26, 2023
@nikclayton nikclayton deleted the v22-with-check-for-updates branch July 26, 2023 14:58
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.

4 participants