Skip to content

Added Support for radar-commons v1.x.x+ with Ktor and Coroutines Integration #442

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
Open
wants to merge 118 commits into
base: master
Choose a base branch
from
Open

Added Support for radar-commons v1.x.x+ with Ktor and Coroutines Integration #442

wants to merge 118 commits into from

Conversation

@Bdegraaf1234
Copy link
Contributor

@Bdegraaf1234 Bdegraaf1234 commented Apr 29, 2024 •
edited by this-Aditya
Loading

Updates to the radar-commons-android library:

  • Added support for radar-commons version 1.x.x and above.
  • Updated the library to utilize Ktor client for network requests.
  • Refactored the code to use coroutines, flows, and channels for better concurrency handling.

@blootsvoets
Expose current source and attributes
c41f935
@blootsvoets
Docs and listen for manual attribute updates
f34c7c3
@blootsvoets
Allow reentrant execution when updating source attributes
b53ab3e
@blootsvoets
Add attributes and source to BaseServiceConnection
c7812e0
@blootsvoets
Added RadarService.flushCaches and misc changes
e6f1ed6
@blootsvoets
Do not include non-plugin directories
9a9d7ed
@blootsvoets
Remove loop in server_status handling
4c93863
@blootsvoets
Add logout handling to all classes
bdf30e7
@blootsvoets
Forward all exceptions to login listeners
92a5b31
@blootsvoets
Build coroutines WIP
053e908
@blootsvoets
Forward logoutSucceeded to listeners
a63db02
@blootsvoets
Remove duplicate logout handling
484269e
@blootsvoets
Full reset after logout and removed deprecated code
cb27886
@blootsvoets
Kotlin readability
92b1647
@blootsvoets
Remove deprecated code and use kotlin-like syntax
eeaa443
@blootsvoets
Misc kotlin readability updates
a8d2302
@blootsvoets
Jwt parser reordering
e4b7125
@blootsvoets
Merge branch 'dynamic_source_coupling' into ktor-coroutines
2e2f7ca
@blootsvoets
WIP
3b75586
@blootsvoets
LiveData transformation improvements
1f294f9
@blootsvoets
Misc changes
ec25bee
@Bdegraaf1234 Bdegraaf1234 self-assigned this Jun 5, 2024
@this-Aditya this-Aditya mentioned this pull request Dec 4, 2024
Closed
@this-Aditya
Modified logs for coroutine task executor
fef8a4c
@this-Aditya
Updated way of starting source managers
55e9810
@this-Aditya
Updated activities, service and serialization
ba40803
@this-Aditya
Updated data handling and sending
b9fb230
@this-Aditya
Updates to bluetooth enforcer
830d7e8
@this-Aditya
Added radar service instance to application class
8cab795
@this-Aditya
Misc changes
4f86d10
@this-Aditya
Now receiving location broadcast via flows
542c276
@this-Aditya
More work on broadcasts
9b626fe
@this-Aditya
Broadcaster deleted
4af35df
@this-Aditya
Removed deprecated startActivityForResult
b4f92b8
@this-Aditya
Misc changes
43167a1
@this-Aditya
Some changes on bluetooth with some minor changes
954ffe8
@this-Aditya
Corrected sensor permissions and config flow collection
5081026
@this-Aditya
Making CoroutineTaskExecutor execution more robust
c292f92
@this-Aditya
Improving chances of task execution by loading them to lifecycleScope
ce8c1ff
@this-Aditya
Making logout succeeded suspend
7953f72
@this-Aditya
Closing resources properly
b9b7f46
@this-Aditya
Changes in sources
b7811bd
@this-Aditya
Logout feature
be2f002
Base automatically changed from dev to master October 23, 2025 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yatharthranjan yatharthranjan Awaiting requested review from yatharthranjan

@peyman-mohtashami peyman-mohtashami Awaiting requested review from peyman-mohtashami

@mpgxvii mpgxvii Awaiting requested review from mpgxvii

At least 1 approving review is required to merge this pull request.

Assignees

@this-Aditya this-Aditya

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@Bdegraaf1234 @pvannierop @yatharthranjan @mpgxvii @this-Aditya @blootsvoets