Skip to content

Conversation

thmsobrmlr
Copy link
Contributor

@thmsobrmlr thmsobrmlr commented Oct 16, 2025

Problem

Follow up to #39711

Changes

  • makes the demo user staff by default
  • uses the system theme for the user

How did you test this code?

Re-created my local environment

@posthog-bot
Copy link
Contributor

Hey @thmsobrmlr! 👋
This pull request seems to contain no description. Please add useful context, rationale, and/or any other information that will help make sense of this change now and in the distant Mars-based future.

@thmsobrmlr thmsobrmlr requested a review from Twixes October 16, 2025 15:12
Copy link
Contributor

@andehen andehen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💖

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

action="store_true",
default=False,
help="Create a staff user",
default=True,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

syntax: "Wether" is misspelled

Suggested change
default=True,
help="Whether the demo user should be a staff user (default: True)",
Prompt To Fix With AI
This is a comment left during a code review.
Path: posthog/management/commands/generate_demo_data.py
Line: 84:84

Comment:
**syntax:** "Wether" is misspelled

```suggestion
            help="Whether the demo user should be a staff user (default: True)",
```

How can I resolve this? If you propose a fix, please make it concise.

@thmsobrmlr thmsobrmlr enabled auto-merge (squash) October 16, 2025 19:55
default=False,
help="Create a staff user",
default=True,
help="Wether the demo user should be a staff user (default: True)",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a typo in the help text: Wether should be Whether.

Suggested change
help="Wether the demo user should be a staff user (default: True)",
help="Whether the demo user should be a staff user (default: True)",

Spotted by Graphite Agent

Fix in Graphite


Is this helpful? React 👍 or 👎 to let us know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants