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.

Sign up for GitHub

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

Jump to bottom

Android 13 and multiple updates #193

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Android 13 and multiple updates #193

wants to merge 22 commits into from

Conversation

TheCataliasTNT2k
Copy link

This PR updates the app to Android 13 (or tries to, I do not have Android 13 and can not test it).
I also took multiple forks and merged their changes into this PR.

tuffnerdstuff and others added 19 commits March 27, 2021 14:03
@tuffnerdstuff
improved docker image rebuild speed
f550c13
@tuffnerdstuff
Android client sends altitude data
6c788e3
@tuffnerdstuff
Backend: push altitude value into points on post
13b6664
@tuffnerdstuff
Frontend: Read and display altitude data
6b556c1
@tuffnerdstuff
Added ALTITUDE_UNIT
8fd96ba
@tuffnerdstuff
hiding all metrics when client is dead
24750a3
@tuffnerdstuff
Refactored LocationUpdatePackage
9cbd1cc
@tuffnerdstuff
Refactored post.php
b5dad6e
@tuffnerdstuff
Moved altitude to its own line and added labels
082da40
@tuffnerdstuff
Displaying metrics is optional
27a1959
@tuffnerdstuff
Hiding altitude if value is null
4ae423a
@tuffnerdstuff
Only unhide altitude if config allows it
33e19cb
Automatically resume shares on device start
e6263e7 
This commit adds an option to automatically resume active shares on device reboot.
Fix sharing not resuming on boot under android 11
00ad4bd 
Due to access restrictions introduced under android
11(https://developer.android.com/guide/components/foreground-services#access-restrictions)
you need an additional permissions to start recieveing location from a
service that wasn't started by a foreground activity
@Bartixxx32
Merge pull request bilde2910#1 from MaeIsBad/master
f33b453 
merge
@Bartixxx32
Merge pull request bilde2910#3 from tuffnerdstuff/feature/altitude
94f8866 
Feature/altitude
@franck-x
build: upgrade gradle to 7.2.2
349c5f2
@franck-x
build: upgrade compileSdkVersion and targetSdkVersion to 33
ff9acda
@franck-x
build: upgrade dependencies
f42949b
@TheCataliasTNT2k
Copy link
Author

Test Android 13 before merge!

@TheCataliasTNT2k
Copy link
Author

Related: #191

@licaon-kter
Copy link
Contributor

licaon-kter commented Apr 2, 2023 •
edited
Loading

The app runs fine on Android 13 from F-Droid.

This PR merges multiple unrelated PRs?

Looks odd...

@TheCataliasTNT2k
Copy link
Author

TheCataliasTNT2k commented Apr 2, 2023 •
edited
Loading

@licaon-kter I rewrote the history a little bit and removed the mess I made while testing.
I know, that the FDroid versions works, but it would be better, if the Google Play version would work as well.
And yes, it somelike "merges multiple PRs" because there were no PRs or I felt like no one cared about them.

@TheCataliasTNT2k
Copy link
Author

I tested it using an emulator with Android 13, but I can not test it on real devices, because I have no devices above Android 11

@2Belette
Copy link

2Belette commented Jun 9, 2023

@TheCataliasTNT2k I am also using Fdroid, would you have an .apk for me to make some test ?

@licaon-kter
Copy link
Contributor

On a quick read it looks nice, hopefully @bilde2910 can take a look too

@TheCataliasTNT2k
Copy link
Author

@TheCataliasTNT2k I am also using Fdroid, would you have an .apk for me to make some test ?

No, sorry, I never built an apk, because unfortunately I do not have any real devices to test it. I only used the Android Emulator....

@licaon-kter licaon-kter mentioned this pull request Aug 7, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@TheCataliasTNT2k @licaon-kter @2Belette @tuffnerdstuff @Bartixxx32 @franck-x