Skip to content

Conversation

odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Sep 2, 2025

What do these changes do?

As suggested by @pcrespov, when an OEC is shown on a Flash Message, if the Support feature is enabled, instead of providing the copy OEC&Context to clipboard button, this PR gives provides the users with a quick way to contact support.

When the user presses the contact support button, a "provide more context" window pops up where the user can provide more context. This is then added to the curated OEC&Context and posted as a new Support conversation.

OEC2SC

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz self-assigned this Sep 2, 2025
@odeimaiz odeimaiz added this to the Cheops milestone Sep 2, 2025
@odeimaiz odeimaiz added t:enhancement Improvement or request on an existing feature a:frontend issue affecting the front-end (area group) labels Sep 2, 2025
Copy link
Contributor

mergify bot commented Sep 2, 2025

🧪 CI Insights

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

🟢 All jobs passed!

But CI Insights is watching 👀

@odeimaiz odeimaiz requested a review from Copilot September 3, 2025 09:12
@odeimaiz odeimaiz marked this pull request as ready for review September 3, 2025 09:13
Copy link
Contributor

@Copilot 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 adds support for contextual chat integration in the OEC (osparc Error Center) flash messages, allowing users to either copy error context or directly open a support chat with the error details.

  • Creates a new FlashMessageOEC class that extends FlashMessage with chat and copy functionality
  • Modifies FlashMessenger to use the new OEC flash message for errors with support IDs
  • Updates support center UI to show "Ask a question" button for all users and changes "Support ID" to "Ticket ID"

Reviewed Changes

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

File Description
FlashMessageOEC.js New flash message component with copy-to-clipboard and chat support functionality
SupportCenter.js Removes conditional display of "Ask a question" button
ConversationPage.js Changes label from "Support ID" to "Ticket ID"
FlashMessenger.js Refactors error handling to use new OEC flash message class and extracts common flash message addition logic

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.

Suggestions:

  • Instead of showing raw JSON, format and send a clear formatted message, for example:
    Hi,
    
    I got this error: {message}
    
    - SupportID: 1234
    - Date: 2023-04-34
    - ...
    
  • After sending the first message, place the cursor in a new draft message so the user can continue writing.
    • Can you delete/edit a sent message? If so the user could also modify the default message
  • Keep in mind: the error could be critical (e.g. a service outage). In that case, the support channel might not be functional. If so, fall back to the previous copy-to-clipboard approach.

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.

Very nice feature!
One proposal, which I think it might improve it's usefulness by a big margin.
In an effort to avoid having no additional context messages, I would ask the user to provide some context message before sending this report over. A popup where they add more context would be great. When they press send on the popup, it will open the support chat with the message already sent (already containing the error).

Copy link
Collaborator

@matusdrobuliak66 matusdrobuliak66 left a comment

Choose a reason for hiding this comment

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

💯

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.

Please check my suggestsions

@odeimaiz odeimaiz requested review from pcrespov and GitHK September 3, 2025 11:19
@odeimaiz
Copy link
Member Author

odeimaiz commented Sep 3, 2025

@pcrespov @GitHK I went for a mix of what you requested. Check the updated description and animation.

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.

very nice! much better.
Thx

Copy link

sonarqubecloud bot commented Sep 3, 2025

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.

Way better! Thanks for enhancing it!

@odeimaiz odeimaiz merged commit 2589946 into ITISFoundation:master Sep 3, 2025
58 checks passed
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) t:enhancement Improvement or request on an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants