Skip to content

Commit

Permalink
show Other/None in actors modal
Browse files Browse the repository at this point in the history
  • Loading branch information
mariusandra committed Mar 21, 2024
1 parent 076732f commit fce1815
Showing 1 changed file with 4 additions and 9 deletions.
13 changes: 4 additions & 9 deletions posthog/hogql_queries/insights/trends/trends_query_runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@
from posthog.hogql.query import execute_hogql_query
from posthog.hogql.timings import HogQLTimings
from posthog.hogql_queries.insights.trends.breakdown_values import (
BREAKDOWN_NULL_NUMERIC_LABEL,
BREAKDOWN_NULL_STRING_LABEL,
BREAKDOWN_OTHER_NUMERIC_LABEL,
BREAKDOWN_OTHER_STRING_LABEL,
Expand Down Expand Up @@ -240,14 +239,10 @@ def to_actors_query_options(self) -> InsightActorsQueryOptionsResponse:
cohort_name = "all users" if str(value) == "0" else Cohort.objects.get(pk=value).name
label = cohort_name
value = value
elif value == BREAKDOWN_OTHER_STRING_LABEL or value == BREAKDOWN_OTHER_NUMERIC_LABEL:
# label = "Other"
# value = BREAKDOWN_OTHER_STRING_LABEL
continue # TODO: Add support for "other" breakdowns
elif value == BREAKDOWN_NULL_STRING_LABEL or value == BREAKDOWN_NULL_NUMERIC_LABEL:
# label = "Null"
# value = BREAKDOWN_NULL_STRING_LABEL
continue # TODO: Add support for "null" breakdowns
elif value == BREAKDOWN_OTHER_STRING_LABEL:
label = "Other (Groups all remaining values)"
elif value == BREAKDOWN_NULL_STRING_LABEL:
label = "None (No value)"
elif is_boolean_breakdown:
label = self._convert_boolean(value)
else:
Expand Down

0 comments on commit fce1815

Please sign in to comment.