[BUG] Can't override get_queryset method when using ObserverModelInstanceMixin and GenericAsyncAPIConsumer

[erdos4d]

[BUG] Can't override get_queryset method when using ObserverModelInstanceMixin and GenericAsyncAPIConsumer

[hishnash]

Hi @erdos4d thank you for reporting this you should absolutly be able to override the get_queryset method... through in my examples I tend to use fitler_queryset (that should work)

just to be clear the intention here is to filter what a user can subscribe to ? or is the intention to filter the results from REST like actions: list retrieve create and update etc?

[hishnash]

So I have done a little more looking into this and the source of the issue is

djangochannelsrestframework/djangochannelsrestframework/observer/generics.py

Line 41 in d6fb7a0

queryset = body.get("queryset", None)

The reason this is here is we need to register a django even hook on the database record. (this needs to happen before any insurance of the consumer is created but rather when django starts up) so we cant use get_queryset as this is an instance method and we need to be able to know the DB record type at django startup to register change handlers on all running threads/instrances of django.

you can use filter_queryset method (as used here https://nilcoalescing.com/blog/BuildingARealtimeSocialNetworkUsingDjangoChannels/#websocket-consumer ) to filter based on the user (however as that post describes this has limited use for subscription in that case you need to follow the suggestions in the post by defining the groups that are used to notify of changes and limiting what groups a user can subscribe to).

[hishnash]

this mixin should realy check if get_queryset has been defined and if it has it should raise some warning.