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
Fix - Email Notes #2356
Fix - Email Notes #2356
Conversation
This pull request is being automatically deployed with Vercel (learn more). docs – ./apps/docs🔍 Inspect: https://vercel.com/cal/docs/Ek3oxHRyuHuWZXzXYWa9zbm7m4xP [Deployment for ed0dcfb canceled] calendso – ./apps/web🔍 Inspect: https://vercel.com/cal/calendso/2SGWtvzvtqKKwJyC5EvUcsrRLDHU |
@@ -530,6 +529,9 @@ export default async function handler(req: NextApiRequest, res: NextApiResponse) | |||
if (rescheduleUid) { | |||
// Use EventManager to conditionally use all needed integrations. | |||
const updateManager = await eventManager.update(evt, rescheduleUid); | |||
// This gets overridden when updating the event - to check if notes have been hidden or not. We just reset this back |
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.
Note at this point it has already been created in prisma with the right values
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.
lgtm but final approval pending from @zomars
This reverts commit 95aa5fe.
What does this PR do?
Currently if you select the option to hide notes. The only way to view them notes is by logging into the app.
It has been requested that the notes still show in the email invitations.
Fixes # (issue)
Type of change
How should this be tested?
Checklist