cal.com icon indicating copy to clipboard operation
cal.com copied to clipboard

[CAL-782] creating an event-type called “book” throws an error (/book is an API endpoint)

Open PeerRich opened this issue 2 years ago • 8 comments
trafficstars

we would need to either re-route it or forbid the use of “book”

preferably the first

From SyncLinear.com | CAL-782

PeerRich avatar Jan 13 '23 19:01 PeerRich

first reported https://twitter.com/codingwithmanny/status/1613844100746199042?s=46&t=fOnOWwxi3zWEejwVizFOaA

PeerRich avatar Jan 13 '23 19:01 PeerRich

Here is the GitStart Ticket for this issue: https://clients.gitstart.com/calcom/1/tickets/CALCOM-6472

gitstart-app[bot] avatar Jan 13 '23 19:01 gitstart-app[bot]

we would need to either re-route it or forbid the use of “book”

preferably the first

From SyncLinear.com | CAL-782

or append the user input with a zero width space if it's called book

michael-azogu avatar Jan 13 '23 19:01 michael-azogu

or append the user input with a zero width space if it's called book

interesting ...

PeerRich avatar Jan 14 '23 00:01 PeerRich

on second thought I don't know about the details. that was just the first thing that popped into my mind

michael-azogu avatar Jan 14 '23 08:01 michael-azogu

If we would have to modify the event name in any way, wouldn't it be better to forbid the use in the first place to make things easier for future use? cc: @PeerRich

gitstart-calcom avatar Jan 16 '23 14:01 gitstart-calcom

i am not sure. would need to pass this question to another dev

PeerRich avatar Jan 16 '23 14:01 PeerRich

discussing now

PeerRich avatar Jan 16 '23 17:01 PeerRich

we'll get rid of the /book route with @JeroenReumkens help

PeerRich avatar Jan 27 '23 10:01 PeerRich

we'll get rid of the /book route with @JeroenReumkens help

I'm working on that BTW @PeerRich https://github.com/calcom/cal.com/pull/6656

zomars avatar Jan 27 '23 18:01 zomars

oh nice!

PeerRich avatar Jan 27 '23 18:01 PeerRich