Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Sep 18, 2025

What do these changes do?

Integrates a 3rd-party booking tool (easyappointments) by adding a new iframe wrapper that is able to handle its styling so it's aligned with the osparc's light and dark themes. It also fills up the user's information (full name and email) to save one step in the appointment booking process.

This feature "Book a Call 3rd" is enabled in development only and it only works if easyappointments is up on http://localhost:8000/booking

BookACall

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz self-assigned this Sep 18, 2025
@mergify
Copy link
Contributor

mergify bot commented Sep 18, 2025

🧪 CI Insights

Here's what we observed from your CI run for b590079.

✅ Passed Jobs With Interesting Signals

Pipeline Job Signal Health on master Retries 🔍 CI Insights 📄 Logs
CI integration-tests Base branch is broken, but the job passed. Looks like this might be a real fix 💪 Broken 0 View View
unit-tests Base branch is broken, but the job passed. Looks like this might be a real fix 💪 Broken 0 View View

@odeimaiz odeimaiz requested a review from Copilot September 18, 2025 16:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Integrates a third-party booking tool by adding a new iframe wrapper and extending the support system to provide an alternative booking interface. This enhancement allows users to access an external booking service directly within the platform.

  • Adds iframe wrapper for external booking service with theme integration
  • Extends support pages with new "Book a Call 3rd" button (development only)
  • Implements stack-based view switching in conversation page for iframe display

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
BookACallIframe.js New iframe wrapper class for external booking service with URL building and theme messaging
HomePage.js Adds development-only third-party booking button with proper event constants
ConversationsPage.js Adds development-only third-party booking button matching HomePage pattern
ConversationPage.js Implements stack container for switching between conversation and iframe views
Conversation.js Adds new system message type constant and fixes default parameter usage

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@odeimaiz odeimaiz changed the title ⚗️ [Frontend] Enh: Integrate the 3rd party Booking tool ⚗️ [Frontend] Integrate the 3rd party Booking tool Sep 18, 2025
@odeimaiz odeimaiz changed the title ⚗️ [Frontend] Integrate the 3rd party Booking tool ⚗️ [Frontend] Integrate the 3rd party Appointment Booking tool Sep 18, 2025
@odeimaiz odeimaiz marked this pull request as ready for review September 18, 2025 16:09
@odeimaiz odeimaiz added this to the Cheops milestone Sep 18, 2025
@odeimaiz odeimaiz added the a:frontend issue affecting the front-end (area group) label Sep 18, 2025
Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh la la !

Copy link
Member

@sanderegg sanderegg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wow

@sonarqubecloud
Copy link

Copy link
Contributor

@GitHK GitHK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess we do not require the phone number to be put in. Will this work if the user has no phone number as well?

@odeimaiz odeimaiz merged commit 5164d46 into ITISFoundation:master Sep 19, 2025
58 checks passed
@matusdrobuliak66 matusdrobuliak66 mentioned this pull request Sep 24, 2025
65 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a:frontend issue affecting the front-end (area group)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants