fix(companion): event type links for org user#5
Conversation
Addresses Cubic AI review feedback (confidence 9/10): The username was hardcoded to 'username' but still used by BasicsTab as a fallback for URL display when bookingUrl is unavailable. This restores the useEffect that fetches the actual username from CalComAPIService.getUsername(). Co-Authored-By: unknown <>
🤖 Augment PR SummarySummary: Updates Companion + extension event-type link handling to work for organization users by relying on API-provided booking URLs. Changes:
Technical Notes: 🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
| const [conferencingOptions, setConferencingOptions] = useState<ConferencingOption[]>([]); | ||
| const [conferencingLoading, setConferencingLoading] = useState(false); | ||
| const [eventTypeData, setEventTypeData] = useState<EventType | null>(null); | ||
| const [bookingUrl, setBookingUrl] = useState<string>(""); |
There was a problem hiding this comment.
bookingUrl is introduced as state, but I don’t see setBookingUrl used anywhere, so it will remain empty and handlePreview/handleCopyLink will always hit the “Booking URL not available” error path. Should this be set from the loaded/saved event type data (e.g., eventTypeData.bookingUrl)?
🤖 Was this useful? React with 👍 or 👎
| const bookingUrl = `https://cal.com/${ | ||
| eventType.users?.[0]?.username || "user" | ||
| }/${eventType.slug}`; | ||
| const bookingUrl = |
There was a problem hiding this comment.
This copy-to-clipboard handler still hardcodes https://cal.com/{username}/{slug} and ignores eventType.bookingUrl, so org/team booking domains can still be copied incorrectly here (unlike the preview handler above).
🤖 Was this useful? React with 👍 or 👎
2 participants
Benchmark PR from agentic-review-benchmarks#5