-
Notifications
You must be signed in to change notification settings - Fork 8
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
💸 Presenter has needlessly nested props #2967
base: main
Are you sure you want to change the base?
Conversation
Coverage report
Test suite run success13 tests passing in 1 suite. Report generated by 🧪jest coverage report action from 06e10e6 |
…nto fix-for-presenter
Deployed changes to https://app-sswwebsite-9eb3-pr-2967.azurewebsites.net |
PageSpeed Insights
View the full report https://pagespeed.web.dev/report?url=https://app-sswwebsite-9eb3-pr-2967.azurewebsites.net |
@amankumarrr is updating the cron job which syncs presenters from people. We would need to run it before merging this so that the Schema gets updated. Unfortunately I can't merge this right now because it would have breaking changes. |
…nto fix-for-presenter
Affected routes:
Fixed 💸 Presenter has needlessly nested props #2839