-
Notifications
You must be signed in to change notification settings - Fork 8.4k
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
perf: Move CRM event creation to tasker #18370
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Skipped Deployments
|
Co-authored-by: CarinaWolli <[email protected]>
…#18587) This reverts commit 4902c6a. Co-authored-by: Alex van Andel <[email protected]>
* fix: disable sending sms when email is present * fix: update test
* fix: error-page.tsx related lint errors * fix: lint no continue-on-error * Adding more lint fixes * Bring back annotate code linting results * Bring back linting continue-on-error * Slimline lint
…#18639) * fix eslint config * fix * add it to dev dep * fix * sync eslint version * force ts-node to compile our ESLint plugin's TS files into CommonJS (which ESLint requires) * fix some lint errors * fix lint errors * remove duplicate classname * make @typescript-eslint/ban-types a warn for packages/trpc files * fix lint errors in trpc * fix lint errors in trpc - 2 * fix * fix * fix lint warnings
Co-authored-by: Alex van Andel <[email protected]>
* potential fix: layout shift * fix lint error
* chore: sync with main * feat: update translations via @replexica --------- Co-authored-by: Replexica <[email protected]>
|
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.
left some comments 🙏🏽
What does this PR do?
https://www.loom.com/share/a8338f139c1c4c8ca524af91f808b531
Mandatory Tasks (DO NOT REMOVE)
How should this be tested?