feat(web): case insensitive slug search

[heuels]

Not sure if this is something you want, but after upgrading we have lots of differently-cased event types that we still need to support 😞

[vercel[bot]]

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

2 Ignored Deployments

Name	Status	Preview	Updated
docs	⬜️ Ignored (Inspect)		Jun 23, 2022 at 1:03PM (UTC)
ui	⬜️ Ignored (Inspect)		Jun 23, 2022 at 1:03PM (UTC)

[vercel[bot]]

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

A member of the Team first needs to authorize it.

[agustif]

I'd go as far to say it might be good to normalize always slugs to be lowercase and use only dash as separator? I know we recently had to cleanup some chars in event-slugs too... Thoughts @zomars ?

[zomars]

@heuels by insensitive you mean 'THIS' === 'this' right?

[heuels]

@zomars yes, that's right!

[zomars]

@heuels @agustif I see this like something desirable yes. But we should make sure it is being used consistently in the codebase. Or find a way to enable this globally:

[agustif]

@zomars we could add maybe a prisma middleware for this, that extends event-type calls adding the slug: { equals: slug, mode: "insensitive" }, where prop?

@heuels if you're up for it, you can checkout how we added soft-delete for bookingReference here https://github.com/calcom/cal.com/blob/main/packages/prisma/middleware/bookingReference.ts A similar approach could work for eventType-slug and maybe others in the future?

[heuels]

@agustif yup, that sounds great! Will get to it later this week 🙌

[hariombalhara]

@heuels Were you done with your changes?

[heuels]

@hariombalhara I have not sadly. We deprioritized using Cal.com for now and this has fallen off my radar. Should I just close this PR?

[emrysal]

This needs to be fixed in the middleware to catch the many, many different places we query things by slug. - further improvement but this will definitely stay on our radar for implementation. Thanks @heuels for the PR and ensuing discussions too.