Skip to content

feat(nodejs): personhog reads#52640

Draft
nickbest-ph wants to merge 14 commits intomasterfrom
nick/nodejs-personhog-person
Draft

feat(nodejs): personhog reads#52640
nickbest-ph wants to merge 14 commits intomasterfrom
nick/nodejs-personhog-person

Conversation

@nickbest-ph
Copy link
Copy Markdown
Contributor

Problem

Changes

How did you test this code?

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

Publish to changelog?

Docs update

@github-actions
Copy link
Copy Markdown
Contributor

🎭 Playwright report · View test results →

⚠️ 4 flaky tests:

  • Editing an insight updates the dashboard tile (chromium)
  • Creating a SQL insight with a variable and overriding it on a dashboard (chromium)
  • Deleting an insight from dashboard redirects back (chromium)
  • Edit mode button enters and exits edit mode (chromium)

These issues are not necessarily caused by your changes.
Annoyed by this comment? Help fix flakies and failures and it'll disappear!

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.

1 participant