-
Notifications
You must be signed in to change notification settings - Fork 916
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
Track More Metrics for Circom with Matomo #5026
Conversation
ioedeveloper
commented
Jul 19, 2024
•
edited
Loading
edited
- Updated path for circom template scripts in other to uniquely identify scripts that are ran for circom.
- Updated e2e tests with new path to fix failing test.
✅ Deploy Preview for remixproject ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -208,6 +208,7 @@ export const createWorkspace = async ( | |||
if (workspaceTemplateName === 'semaphore' || workspaceTemplateName === 'hashchecker' || workspaceTemplateName === 'rln') { | |||
const isCircomActive = await plugin.call('manager', 'isActive', 'circuit-compiler') | |||
if (!isCircomActive) await plugin.call('manager', 'activatePlugin', 'circuit-compiler') | |||
_paq.push(['trackEvent', 'circuit-compiler', 'template', 'create', workspaceTemplateName]) |
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.
are we generally using 5 params with _paq.push
?
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.
No, i think you can use as many