Fix URL by removing slash and backslash

[Jenietoc]

What does this PR do?

This PR validates that when creating or editing an eventType it is not possible to write slash or backslash.

Fixes #1729

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How should this be tested?

• This implementation was tested by creating and editing several newTypes and the validation worked correctly.

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code and corrected any misspellings
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

[vercel]

@Jenietoc is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

[zomars]

Left some comments, we should also avoid unsanitized slugs to reach the DB by updating our schema here:

cal.com/prisma/zod-utils.ts

Line 9 in 38b41f0

export const eventTypeSlug = z.string().transform((val) => val.trim());

To this:

export const eventTypeSlug = z.string().transform((val) => slugify(val.trim()));

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

docs – ./apps/docs

🔍 Inspect: https://vercel.com/cal/docs/EuRBtwCtkuD36XoVPeJYN3Kh4KBq
✅ Preview: Canceled

[Deployment for 77c1390 canceled]

calendso – ./apps/web

🔍 Inspect: https://vercel.com/cal/calendso/AZnoauCGzpdPCnmDVitepun9CsPU
✅ Preview: https://calendso-git-fork-kommitters-fix-url-problem-cal.vercel.app

[PeerRich]

hey @Jenietoc, there seems to be conflicts that only the PR creator can fix "This branch has conflicts that must be resolved"

can you also take a look at @zomars comment: #1733 (review)?

[Jenietoc]

hey @Jenietoc, there seems to be conflicts that only the PR creator can fix "This branch has conflicts that must be resolved"

can you also take a look at @zomars comment: #1733 (review)?

Hi @PeerRich conflicts have already been resolved

[PeerRich]

we moved to turbo repo which caused this branch to fail @Jenietoc. can you take a look at the new file structure? apps/web

[Jenietoc]

we moved to turbo repo which caused this branch to fail @Jenietoc. can you take a look at the new file structure? apps/web

I think I have already updated the new project structure.

[zomars]

There's seems to be an issue with the classname sorted. We may want to prioritize this #1796

[Jenietoc]

Hello @PeerRich @zomars , what should I do to fix the errors in this PR? and if you can do merge.

[PeerRich]

Hello @PeerRich @zomars , what should I do to fix the errors in this PR? and if you can do merge.

@zomars will review and likely merge today 🙏

[zomars]

Hello @PeerRich @zomars , what should I do to fix the errors in this PR? and if you can do merge.

Hey @Jenietoc please rebase/merge from main and solve a conflict with packages/prisma/zod-utils.ts I tried to solved it for you but it seems that your settings doesn't allow me to.

[Jenietoc]

Hello @PeerRich @zomars , what should I do to fix the errors in this PR? and if you can do merge.

Hey @Jenietoc please rebase/merge from main and solve a conflict with packages/prisma/zod-utils.ts I tried to solved it for you but it seems that your settings doesn't allow me to.

I will talk to the person who created the fork to give those permissions, I will also try to solve the conflicts. thank you very much for your time.

[zomars]

Merging as it already passed but it got out of date again.