-
Notifications
You must be signed in to change notification settings - Fork 7.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add report bug buttons #11304
feat: Add report bug buttons #11304
Conversation
Co-authored-by: Cornelius Suermann <[email protected]>
@@ -36,6 +43,10 @@ function onZoomOut() { | |||
function onZoomToFit() { | |||
emit('zoom-to-fit'); | |||
} | |||
|
|||
function trackBugReport() { | |||
telemetry.track('User clicked bug report button in canvas', {}, { withPostHog: true }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add event to track when this is used. Sending to PH will make it easy to see the issue for Cloud users and reproduce exact issue.
@@ -14,7 +14,7 @@ | |||
"format:check": "biome ci . && prettier --check . --ignore-path ../../.prettierignore", | |||
"serve": "cross-env VUE_APP_URL_BASE_API=http://localhost:5678/ vite --host 0.0.0.0 --port 8080 dev", | |||
"test": "vitest run", | |||
"test:dev": "vitest" | |||
"test:dev": "vitest --silent=false" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allows console logs to show up when writing unit tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected. Thank you for doing of this!
✅ All Cypress E2E specs passed |
Got released with |
Summary
Add bug reporting buttons to new canvas and sidebar
Related Linear tickets, Github issues, and Community forum posts
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)