Skip to content

Conversation

@lricoy
Copy link
Member

@lricoy lricoy commented Dec 5, 2025

Problem

I didn't toggle those others to true due to the linting not recognizing null as falsy (!!!???) and it should have done it on #42455

Changes

Actually toggle the canOpenModal to true again as the default option.

How did you test this code?

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

Changelog: (features only) Is this feature complete?

@lricoy lricoy requested a review from a team December 5, 2025 22:57
@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2025

Size Change: 0 B

Total Size: 3.51 MB

ℹ️ View Unchanged
Filename Size
frontend/dist/toolbar.js 3.51 MB

compressed-size-action

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, no comments

Edit Code Review Agent Settings | Greptile

@lricoy lricoy enabled auto-merge (squash) December 5, 2025 23:10
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