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

Published 20 hours ago • verified publisher icon calcom

[CAL-2618] Add tests to `handleSeats`

Open joeauyeung opened this issue 1 year ago • 7 comments

This will require a refactor of handleNewBooking and handleSeats as both functions are tied very closely together.

From SyncLinear.com | CAL-2618

joeauyeung avatar Oct 18 '23 13:10 joeauyeung

can you assign me this issue also please explain a little more.

pankajsajwan12 avatar Oct 18 '23 17:10 pankajsajwan12

can you assign me this issue

jayam30 avatar Oct 20 '23 22:10 jayam30

what exactly is the change that we need to make?

Rohxn16 avatar Oct 22 '23 02:10 Rohxn16

Can please you assign this for me?

Jayanth-Kammela avatar Oct 22 '23 04:10 Jayanth-Kammela

handleSeats is a mammoth 600 lines function. Deserves its own file and test suite.

codesmith922 avatar Oct 24 '23 11:10 codesmith922

can you please elaborate on the issue

AnuragM23 avatar Oct 26 '23 18:10 AnuragM23

@joeauyeung can this be closed with https://github.com/calcom/cal.com/pull/12905?

CarinaWolli avatar Jan 16 '24 18:01 CarinaWolli