Skip to content

fix: add team model in prisma middleware#3947

Merged
zomars merged 19 commits intocalcom:mainfrom
Udit-takkar:fix/team-description
Aug 26, 2022
Merged

fix: add team model in prisma middleware#3947
zomars merged 19 commits intocalcom:mainfrom
Udit-takkar:fix/team-description

Conversation

@Udit-takkar
Copy link
Copy Markdown
Contributor

Fixes: #3940
There were some changes made in prisma middleware to sanitise event description. user event type description were handled but /team/slug/type was using different query.

Screenshot 2022-08-24 at 10 18 56 PM

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 24, 2022

@Udit-takkar is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown
Contributor

@zomars zomars left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch! Let's address some issues first tho 🙏🏽

Comment thread packages/prisma/middleware/eventTypeDescriptionParseAndSanitize.ts Outdated
@Udit-takkar Udit-takkar requested a review from zomars August 25, 2022 19:16
@PeerRich PeerRich added ♻️ autoupdate tells kodiak to keep this branch up-to-date automerge labels Aug 25, 2022
Copy link
Copy Markdown
Contributor

@zomars zomars left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution 🙏

Comment thread packages/prisma/middleware/eventTypeDescriptionParseAndSanitize.ts Outdated
@Udit-takkar Udit-takkar requested a review from zomars August 26, 2022 10:04
@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 26, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Aug 26, 2022 at 6:47PM (UTC)
1 Ignored Deployment
Name Status Preview Updated
swagger ⬜️ Ignored (Inspect) Aug 26, 2022 at 6:47PM (UTC)

@zomars zomars merged commit 2d3a6fb into calcom:main Aug 26, 2022
@zomars
Copy link
Copy Markdown
Contributor

zomars commented Aug 26, 2022

Thank you for your contribution 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge ♻️ autoupdate tells kodiak to keep this branch up-to-date

Projects

None yet

Development

Successfully merging this pull request may close these issues.

description missing on team events

3 participants