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.

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

Support for Android Paging 3 #913

Open
RobbWatershed opened this issue Jul 23, 2020 · 8 comments
Open

Support for Android Paging 3 #913

RobbWatershed opened this issue Jul 23, 2020 · 8 comments

Comments

@RobbWatershed
Copy link
Contributor

https://developer.android.com/topic/libraries/architecture/paging/v3-overview

Not really an urgency as it is still in alpha phase, but definitely something to plan for the middle term imho.

@RobbWatershed
Copy link
Contributor Author

FYI, Paging 3 stable has been released this week ! 🙃

https://developer.android.com/jetpack/androidx/releases/paging#3.0.0

@RobbWatershed
Copy link
Contributor Author

@mikepenz is there any honest chance we're getting this, or is that help wanted tag an actual PR request to a fellow enlightened coder ?

Forgive me for insisting, but I have high hopes paging 3 will solve that nasty "out of bounds" issue I've been having with paging 2

https://issuetracker.google.com/issues/135628748#comment10

https://stackoverflow.com/questions/62860503/android-paging-library-index-out-of-bound-exception

(plus not supporting Paging 3 will probably be the slow death of FastAdapter in favour of younger libs that have been built with it... which would be really sad)

@mikepenz
Copy link
Owner

@RobbWatershed yes you are right, the help wanted flag indeed means that I would appreciate this being contributed by a fellow coder.

As you note slow death. Ultimately with the stable state of Compose, any adapter library is now deprecated on the long term. Compose's LazyList just functions very different compared to the RV.
Also the official adapter for RV with the ConcatAdapter cloning some of the FastAdapters feature is also interesting.

As a result to those things, the library is now mainly in support mode to ensure existing apps won't have issues with it. Still happy though to take PRs to improve things, like paging as you noted.

@RobbWatershed
Copy link
Contributor Author

@mikepenz thanks for the information ! I might give it a try myself, actually.

Before I start, from an architectural point of view, what would be the best way to do that in your opinion ?

  • Upgrade the paged extension to paging 3 and break compatibility with paging 2 ?
  • Create a separate paged3 extension that works with paging 3 ?
  • A third option ?

@mikepenz
Copy link
Owner

@RobbWatershed I believe as paging 2 is kinda deprecated (and the dependency would still exist in older FastAdapter versions) the best approach will be to upgrade the paged extension to paging 3 and break compatibility.

People who still needs paging 2 can still get current FA versions which are stable

@vladdovgal
Copy link

Hi @mikepenz, @RobbWatershed
Are there any updates on supporting Paging 3, or is it out of plans?

@mikepenz
Copy link
Owner

@vladdovgal it would be a great addition, however, time is currently very limited. It won't be anything I can work on.

Looking for contributors who may would want to contribute this expansion.

@RobbWatershed
Copy link
Contributor Author

RobbWatershed commented Nov 23, 2023

Good news is I've gotten way better at Kotlin since the moment I posted this issue

Not-so-good news is I have the same problem as Mike, i.e. available time + I don't know the inner workings / design of FastAdapter as well as the core team

=> Don't actively count on me, even if the chances of me contributing are still > 0 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants