-
Notifications
You must be signed in to change notification settings - Fork 1
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 telemetry #12
Conversation
frontend/src/index.tsx
Outdated
ReactDOM.render( | ||
<React.StrictMode> | ||
<QueryClientProvider client={queryClient}> | ||
<HashRouter> | ||
<CommonParamsProvider> | ||
<App /> | ||
</CommonParamsProvider> | ||
</HashRouter> | ||
</QueryClientProvider> | ||
</React.StrictMode>, | ||
document.getElementById("root") | ||
<React.StrictMode> | ||
<QueryClientProvider client={queryClient}> | ||
<HashRouter> | ||
<CommonParamsProvider> | ||
<App /> | ||
</CommonParamsProvider> | ||
</HashRouter> | ||
</QueryClientProvider> | ||
</React.StrictMode>, | ||
document.getElementById('root'), | ||
); |
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.
oops. will revert
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.
this was caused by the pre-commit hook which hasn't been run on existing code.
frontend/tsconfig.json
Outdated
"compilerOptions": { | ||
"baseUrl": "./src", | ||
"target": "es5", | ||
"lib": ["dom", "dom.iterable", "esnext"], | ||
"allowJs": true, | ||
"skipLibCheck": true, | ||
"esModuleInterop": true, | ||
"allowSyntheticDefaultImports": true, | ||
"strict": true, | ||
"forceConsistentCasingInFileNames": true, | ||
"noFallthroughCasesInSwitch": true, | ||
"module": "esnext", | ||
"moduleResolution": "nodenext", | ||
"resolveJsonModule": true, | ||
"isolatedModules": true, | ||
"noEmit": true, | ||
"jsx": "react-jsx" | ||
}, | ||
"include": ["src"] | ||
} |
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.
will revert. my bad
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.
this was caused by the pre-commit hook which hasn't been run on existing code.
had to disable pre-commit hooks which change styles to keep my diff appropriate.
22873f1
to
d40917a
Compare
@laurentsenta when will this be deployed once merged |
@SgtPooki should be instant, this was hooked into a fleek project. Feel free to merge whenever. |
fixes #10