Skip to content
This repository has been archived by the owner on Nov 12, 2024. It is now read-only.

Migrate remaining screens to Compose #719

Merged
merged 31 commits into from
Nov 23, 2020
Merged

Migrate remaining screens to Compose #719

merged 31 commits into from
Nov 23, 2020

Conversation

chrisbanes
Copy link
Owner

@chrisbanes chrisbanes commented Oct 28, 2020

The remaining screens are all AndroidX Paging based, and some use grids.

  • Convert Followed to Compose
  • Convert Watched to Compose
  • Convert Trending to Compose
  • Convert Popular to Compose
  • Convert Recommended to Compose

@chrisbanes chrisbanes changed the title Start of migration to Paging 3 Migrate to Paging 3 Nov 3, 2020
chrisbanes and others added 22 commits November 3, 2020 16:34
* [Watched] Copy followed changes to watched

* Finish off WatchedViewModel

* Make SubjectInteractor have a replay of 1

To make it work more ConflatedBroadcastChannel

* Update watched list item layout
* Add working Trending shows
* Add a fake Lazy Grid implementation
* Try using RemoteMediator
* Fix refreshing for Trending shows
* Exploring Andrey's paging CL
* More explorations with paging
* Copy over new functions from compose-paging alpha02
* Use LazyPagingItems.refresh()
* Remove invoke() overloads
* Incremental composition is mostly working
* Use forEach rather than asFlow().collect {}
* Add placeholders to Trending
* Workaround paging3 issues
* Add more docs to LazyPaging.kt
* Remove logging from UpdateTrendingShows
@chrisbanes chrisbanes marked this pull request as ready for review November 23, 2020 16:23
@chrisbanes chrisbanes changed the title Migrate to Paging 3 Migrate remaining screens to Compose Nov 23, 2020
@chrisbanes chrisbanes merged commit 1c69c28 into main Nov 23, 2020
@chrisbanes chrisbanes deleted the cb/paging3 branch November 23, 2020 16:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants