fix: ternary operator with lenght causing 0

[PeerRich]

PSA: dont use length with && to prevent trailing 0's

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
api	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 12, 2023 0:12am
cal	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 12, 2023 0:12am
cal-staging	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 12, 2023 0:12am
ui	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 12, 2023 0:12am

[github-actions]

Thank you for following the naming conventions! 🙏

[alwaysmeticulous]

🤖 Meticulous spotted visual differences in 67 of 259 screens tested: view and approve differences detected.

_{Last updated for commit ce6bdee. This comment will update as new commits are pushed.}

[github-actions]

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

[deploysentinel]

Current Playwright Test Results Summary

✅ 88 Passing - ⚠️ 3 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 07/12/2023 12:16:35pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: ce6bdee

Started: 07/12/2023 12:14:20pm UTC

⚠️ Flakes

📄   apps/web/playwright/event-types.e2e.ts • 1 Flake

Test Case Results

Test Case	Last 7 days Failures	Last 7 days Flakes
Event Types tests user can add multiple organizer address Retry 1 • Initial Attempt	0% (0) 0 / 42 runs failed over last 7 days	11.90% (5) 5 / 42 runs flaked over last 7 days

---

📄   packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 2 Flakes

Top 1 Common Error Messages

null	2 Test Cases Affected

Test Case Results

Test Case	Last 7 days Failures	Last 7 days Flakes
Popup Tests should be able to reschedule Retry 1 • Initial Attempt	3.70% (1) 1 / 27 run failed over last 7 days	96.30% (26) 26 / 27 runs flaked over last 7 days
Popup Tests should open Routing Forms embed on click Retry 1 • Initial Attempt	0% (0) 0 / 27 runs failed over last 7 days	14.81% (4) 4 / 27 runs flaked over last 7 days

---

View Detailed Build Results

---

[emrysal]

!! works too and is cleaner 👍

[odicho]

I think a cleaner approach here is to actually do a strict comparison to 0, !== 0 or > 0. I try to avoid the ? <> : null syntax whenever possible.