Add reviews support #1534
Add reviews support #1534
Conversation
|
This has been fully tested to work on mobile, still need to add UI for TV though. |
library/src/commonMain/kotlin/com/lagradost/cloudstream3/MainAPI.kt
Outdated
Show resolved
Hide resolved
library/src/commonMain/kotlin/com/lagradost/cloudstream3/MainAPI.kt
Outdated
Show resolved
Hide resolved
library/src/commonMain/kotlin/com/lagradost/cloudstream3/MainAPI.kt
Outdated
Show resolved
Hide resolved
| open suspend fun loadReviews( | ||
| url: String, | ||
| page: Int, | ||
| showSpoilers: Boolean = false |
There was a problem hiding this comment.
While this api might be functional, we might want to also add sort by or filter by.
There was a problem hiding this comment.
I gave spoiler support control to the API so it is no longer done here at all, can pass isSpoiler to UserReview and then it has a UI for spoilers (which will be improved) as for sort I guess we could add it if you think we need to?
app/src/main/java/com/lagradost/cloudstream3/ui/APIRepository.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/com/lagradost/cloudstream3/ui/result/ReviewAdapter.kt
Outdated
Show resolved
Hide resolved
library/src/commonMain/kotlin/com/lagradost/cloudstream3/MainAPI.kt
Outdated
Show resolved
Hide resolved
| url: String, | ||
| page: Int, | ||
| showSpoilers: Boolean = false | ||
| ): List<UserReview> { |
There was a problem hiding this comment.
Unlike HomePageResponse, this lacks a hasNext making it hard to know if we should re-fetch or not if it fails.
app/src/main/java/com/lagradost/cloudstream3/ui/result/ResultViewModel2.kt
Outdated
Show resolved
Hide resolved
|
I put this on hold for now. I am go back to it eventually but this has become much more work than I originally bargained for... |
TODO: