MerginMaps / mobile

Mobile app for Mergin Maps 📲. Open QGIS projects and manage vector data on your phone, even offline 📶 (Android/iOS/Windows app)
https://merginmaps.com
GNU General Public License v2.0
280 stars 62 forks source link

Android: fused location provider #3447

Closed wonder-sk closed 3 weeks ago

wonder-sk commented 1 month ago

Initial version of the support for fused location provider:

Default (fused available) Default (fused not available) Using fused
Screenshot_20240604_145711 Screenshot_20240604_145022 Screenshot_20240604_145720
github-actions[bot] commented 1 month ago

Pull Request Test Coverage Report for Build 9368402942

Details


Files with Coverage Reduction New Missed Lines %
input/app/featuresmodel.cpp 1 67.8%
input/core/merginuserinfo.cpp 3 73.76%
input/app/position/providers/positionprovidersmodel.cpp 31 64.81%
input/app/position/positionkit.cpp 37 80.19%
<!-- Total: 72 -->
Totals Coverage Status
Change from base Build 9351292521: 0.04%
Covered Lines: 7674
Relevant Lines: 12800

💛 - Coveralls
github-actions[bot] commented 1 month ago

Pull Request Test Coverage Report for Build 9368402942

Details


Files with Coverage Reduction New Missed Lines %
input/app/featuresmodel.cpp 1 67.8%
input/core/merginuserinfo.cpp 3 73.76%
input/app/position/providers/positionprovidersmodel.cpp 31 64.81%
input/app/position/positionkit.cpp 37 80.19%
<!-- Total: 72 -->
Totals Coverage Status
Change from base Build 9351292521: 0.04%
Covered Lines: 7674
Relevant Lines: 12800

💛 - Coveralls
github-actions[bot] commented 1 month ago

Pull Request Test Coverage Report for Build 9368402942

Details


Files with Coverage Reduction New Missed Lines %
input/app/featuresmodel.cpp 1 67.8%
input/core/merginuserinfo.cpp 3 73.76%
input/app/position/providers/positionprovidersmodel.cpp 31 64.81%
input/app/position/positionkit.cpp 37 80.19%
<!-- Total: 72 -->
Totals Coverage Status
Change from base Build 9351292521: 0.04%
Covered Lines: 7674
Relevant Lines: 12800

💛 - Coveralls
wonder-sk commented 3 weeks ago

@tomasMizera hopefully the last bits should be sorted

github-actions[bot] commented 3 weeks ago

Pull Request Test Coverage Report for Build 9437629663

Details


Files with Coverage Reduction New Missed Lines %
input/core/merginuserinfo.cpp 3 73.76%
input/app/position/providers/positionprovidersmodel.cpp 31 64.81%
input/app/position/positionkit.cpp 37 80.19%
<!-- Total: 71 -->
Totals Coverage Status
Change from base Build 9351292521: 0.005%
Covered Lines: 7669
Relevant Lines: 12800

💛 - Coveralls
github-actions[bot] commented 3 weeks ago

Pull Request Test Coverage Report for Build 9437629663

Details


Files with Coverage Reduction New Missed Lines %
input/core/merginuserinfo.cpp 3 73.76%
input/app/position/providers/positionprovidersmodel.cpp 31 64.81%
input/app/position/positionkit.cpp 37 80.19%
<!-- Total: 71 -->
Totals Coverage Status
Change from base Build 9351292521: 0.005%
Covered Lines: 7669
Relevant Lines: 12800

💛 - Coveralls
github-actions[bot] commented 3 weeks ago

Pull Request Test Coverage Report for Build 9437629663

Details


Files with Coverage Reduction New Missed Lines %
input/core/merginuserinfo.cpp 3 73.76%
input/app/position/providers/positionprovidersmodel.cpp 31 64.81%
input/app/position/positionkit.cpp 37 80.19%
<!-- Total: 71 -->
Totals Coverage Status
Change from base Build 9351292521: 0.005%
Covered Lines: 7669
Relevant Lines: 12800

💛 - Coveralls
tomasMizera commented 3 weeks ago

CI failures are unrelated

jozef-budac commented 1 week ago

Verified, I can see the fused location provided according to the screenshot. I also verified that I can switch to it and that the application remembers the user's preference.