-
Notifications
You must be signed in to change notification settings - Fork 0
fix(companion): event type links for org user #26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: augment_combined_20260121_augment_sentry_coderabbit_1_base_fixcompanion_event_type_links_for_org_user_pr706
Are you sure you want to change the base?
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 the Companion app and browser extension to use an API-provided Changes:
Technical Notes: This relies on API V2 returning a full booking URL per event type; UI/extension behavior now depends on that field being present. 🤖 Was this summary useful? React with 👍 or 👎 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bookingUrl is introduced as state here but never set anywhere (no setBookingUrl(...) usage), so handlePreview/handleCopyLink will always hit the "Booking URL not available" error path and BasicsTab will never receive the real URL.
Other Locations
companion/app/(tabs)/(event-types)/event-type-detail.tsx:949companion/app/(tabs)/(event-types)/event-type-detail.tsx:957companion/app/(tabs)/(event-types)/event-type-detail.tsx:1430
🤖 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Benchmark PR from qodo-benchmark#706