dimagi / django-cte

Common Table Expressions (CTE) for Django
Other
334 stars 46 forks source link

Fix OuterRef resolution in CTE subquery #16

Closed SebCorbin closed 4 years ago

SebCorbin commented 4 years ago

@millerdev Hey, I tackled this problem by implementing a CTESubquery class that also resolve expressions in _with_cte

millerdev commented 4 years ago

Thanks for your contribution @SebCorbin. I will try to find some time to review it soon.

millerdev commented 4 years ago

@SebCorbin I would prefer not to add another public CTE* class if possible. I found a way to get around that in #19. However, the effort you put into this was very helpful. Thank you!

SebCorbin commented 4 years ago

Nice, that’s an even better solution!