You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With a StreamedPaginatedListMixin based Consumer, the response is broken up into pages, but the server does not wait for us to request pages. Therefore, to accommodate listing a StreamedPaginatedListMixin based consumer, the list operation should take a callback, similar to subscribe, where the library user can handle one page at a time. Additionally, we may need users, or djangochannelsrestframework, to have a custom paginator that returns the expected number of pages, so dcrf-client knows when to clean up the listener.
Listing a PaginatedListMixin based Consumer should work with dcrf-client as is, although it leaves going through all the pages to the library user. The listing callback could provide a function for library users to get the next page if there is one.
The text was updated successfully, but these errors were encountered:
A merged
djangochannelsrestframework
pull request, NilCoalescing/djangochannelsrestframework#71, adds pagination support. It would be nice ifdcrf-client
got updated to provide handling of this. (Also, see NilCoalescing/djangochannelsrestframework#76)With a
StreamedPaginatedListMixin
basedConsumer
, the response is broken up into pages, but the server does not wait for us to request pages. Therefore, to accommodate listing aStreamedPaginatedListMixin
based consumer, the list operation should take a callback, similar to subscribe, where the library user can handle one page at a time. Additionally, we may need users, ordjangochannelsrestframework
, to have a custom paginator that returns the expected number of pages, sodcrf-client
knows when to clean up the listener.Listing a
PaginatedListMixin
basedConsumer
should work withdcrf-client
as is, although it leaves going through all the pages to the library user. The listing callback could provide a function for library users to get the next page if there is one.The text was updated successfully, but these errors were encountered: