Skip to content

Commit

Permalink
Return conversion for PAGE queries
Browse files Browse the repository at this point in the history
Simply use the main query, scroll bounce/bounce doesn't make sense
  • Loading branch information
rafaeelaudibert committed Dec 5, 2024
1 parent d24dfc5 commit ec5cc09
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion posthog/hogql_queries/web_analytics/stats_table.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,9 @@ def __init__(self, *args, **kwargs):

def to_query(self) -> ast.SelectQuery:
if self.query.breakdownBy == WebStatsBreakdown.PAGE:
if self.query.includeScrollDepth and self.query.includeBounceRate:
if self.query.conversionGoal:
return self.to_main_query(self._counts_breakdown_value())
elif self.query.includeScrollDepth and self.query.includeBounceRate:
return self.to_path_scroll_bounce_query()
elif self.query.includeBounceRate:
return self.to_path_bounce_query()
Expand Down

0 comments on commit ec5cc09

Please sign in to comment.