cashapp / multiplatform-paging

A library that packages AndroidX Paging for Kotlin/Multiplatform.
Apache License 2.0
567 stars 28 forks source link

kotlin.NotImplementedError: An operation is not implemented: onChanged(position=10, count=10, payload=null) #191

Open kalki-v-10464 opened 11 months ago

kalki-v-10464 commented 11 months ago

Team, I'm encountering this crash issue in the iOS app, even though the paging library (version 3.3.0-alpha02-0.4.0) works fine on Android. I've attached a screenshot of the error for reference. Could someone please investigate and provide assistance with resolving this?

kotlin.NotImplementedError: An operation is not implemented: onChanged(position=10, count=10, payload=null)

Screenshot 2023-12-18 at 12 12 20 PM
veyndan commented 10 months ago

It looks like you hit this TODO block!

https://github.com/cashapp/multiplatform-paging/blob/f7b48b3545ab6ea9906894acb261aa9f956bef8c/paging-runtime-uikit/src/iosMain/kotlin/app/cash/paging/PagingCollectionViewController.kt#L54

This should really be replaced with a proper implementation. I'll have a look at implementing it when I'm back from vacation (I'm off until early January).

kalki-v-10464 commented 9 months ago

Hello @veyndan, any update on this?

veyndan commented 8 months ago

Sorry for the delay on this.

In the next version of AndroidX Paging (3.3.0-alpha03), they completely revamped the diffing mechanism, so much of this will change. I'm planning on tracking the next version of AndroidX Paging within a week or two (my current computer is broken — in the midst of getting a new one), so I'll push out a fix for this when I track the next AndroidX Paging release.