Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Sep 22, 2025

What do these changes do?

A bug was introduced in #8393, not letting users interact with the workbench. This PR fixes it.

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz self-assigned this Sep 22, 2025
@odeimaiz odeimaiz added this to the Cheops milestone Sep 22, 2025
@odeimaiz odeimaiz added bug buggy, it does not work as expected a:frontend issue affecting the front-end (area group) labels Sep 22, 2025
@mergify
Copy link
Contributor

mergify bot commented Sep 22, 2025

🧪 CI Insights

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

✅ Passed Jobs With Interesting Signals

Pipeline Job Signal Health on master Retries 🔍 CI Insights 📄 Logs
CI system-tests Base branch is healthy, but retries were needed. Could be early signs of flakiness 👀 Healthy 1 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 changed the title 🐛 [Frontend] RTC fixes 🐛 [Frontend] Fixe workbench Sep 22, 2025
@odeimaiz odeimaiz changed the title 🐛 [Frontend] Fixe workbench 🐛 [Frontend] Fix workbench Sep 22, 2025
@odeimaiz odeimaiz marked this pull request as ready for review September 22, 2025 13:09
@sonarqubecloud
Copy link

@odeimaiz odeimaiz requested a review from Copilot September 22, 2025 13:11
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

This PR fixes a workbench interaction bug that was introduced in PR #8393 by removing a decorator property and corrects button text/icons in support pages.

  • Removes the decorator: "rounded" property from NodeUI configuration that was causing workbench interaction issues
  • Updates "Book a Call 3rd" button text to "Book a Call" and changes icon from phone to flask in support pages

Reviewed Changes

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

File Description
NodeUI.js Removes decorator property to fix workbench interaction bug
HomePage.js Updates button text and icon for book-a-call functionality
ConversationsPage.js Updates button text and icon for book-a-call functionality

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

@odeimaiz odeimaiz enabled auto-merge (squash) September 22, 2025 13:29
@odeimaiz odeimaiz merged commit 562497c into ITISFoundation:master Sep 22, 2025
108 of 110 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) bug buggy, it does not work as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants