Bug in MSSQL driver (more than 2000 approvals) about django-river

[JohnieBraaf]

I've written a simple fix for the above described issue, by pushing down the workflow object id into the query.
Please see commit: JohnieBraaf@58f4962
I see that I have left some debugging code, so please ignore that.

Let me know what you think @javrasya

[javrasya]

Hi @JohnieBraaf. Thanks for reporting this and for the PR. I think this is a bug that should definitely be fixed probably in a similar way you did in your PR.

The thing is that the get_available_approvals method is an abstract method that both MSSQL and other DB versions are implementing. So we can't just update it for the MSSQL one.

The other thing is that this method is for fetching all available approvals for a user on every object but also can be used for a single workflow object. In such case I also think the object id should be passed down instead of applying the ORM filter on top of that (in here).

I will try to drop some comments on your changes. Let's try to discuss it there. Maybe even initialize a PR to have the conversation there.

[JohnieBraaf]

Thnx @javrasya for your feedback, I will have a look