Implement `Pager.refresh` for JVM target

kuuuurt / multiplatform-paging

Kotlin Multiplatform library for Pagination

Apache License 2.0

251 stars 22 forks source link

Implement `Pager.refresh` for JVM target #26

Closed msfjarvis closed 2 years ago

msfjarvis commented 2 years ago

Fixes #22

Builds upon https://github.com/kuuuurt/multiplatform-paging/commit/aed497443bd35ec451583db0395437ed2df4de4d to make the refresh API available on JVM as well.

msfjarvis commented 2 years ago

This seems to have broken between now and the last time I was working on this code, converting to draft while I investigate.

t7costa commented 2 years ago

What was the issue here? I'm also interested in this behavior...

msfjarvis commented 2 years ago

What was the issue here? I'm also interested in this behavior...

The proper fix required exposing AndroidX Paging key creation semantics in the public API which I could not figure out how to do correctly. I ended up switching to moko-paging after adding a JVM target there.