Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Sep 12, 2025

What do these changes do?

Feedback provided by @calys

  • NavBar -> Help Support (1 click)
  • Book a call button in home page
  • Remove intro texts
  • System icebreaking messages for better conversation initiation: different messages for Ask a Question and Book a Call
  • And follow-up message after first message is sent by the user
  • Book a Call, remove set appointment logic, stick to simple conversation. But add a first message containing the "Book a Call" message (this goes to Fogbugz) and also rename the conversation
  • Ticket ID: make uuid shorter
  • Let supporters rename the conversation (backend still missing)
  • Re-support conversation link coming from fogbugz
  • User details: Select and copy the field values

User:
SC_Feedback

Support member:
SC_Feedback_B

Related issue/s

How to test

Dev-ops

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

mergify bot commented Sep 12, 2025

🧪 CI Insights

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

✅ Passed Jobs With Interesting Signals

Pipeline Job Signal Health on master Retries 🔍 CI Insights 📄 Logs
CI system-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 15, 2025 13:41
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 implements a comprehensive frontend support center feedback system with streamlined conversation handling. The changes focus on improving the user experience for creating and managing support conversations through a more unified interface.

Key Changes

  • Replaces complex conversation creation logic with a simplified createConversation pattern that handles different conversation types (Ask a Question, Book a Call, Report OEC)
  • Introduces system messages for better conversation initiation and follow-up messaging
  • Adds UI enhancements including selectable labels in user profiles, shortened UUID display for ticket IDs, and improved conversation management buttons

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
osparc/utils/Utils.js Adds uuidToShort utility function for displaying shortened UUIDs
osparc/user/UserProfile.js Makes user profile labels selectable by centralizing label creation
osparc/ui/message/FlashMessageOEC.js Updates OEC error reporting to use new conversation creation pattern
osparc/support/SupportCenter.js Refactors conversation handling to use unified createConversation method
osparc/support/HomePage.js Redesigns home page with side-by-side conversation buttons and removes intro text
osparc/support/ConversationsPage.js Simplifies conversations page by removing intro text and using unified conversation creation
osparc/support/Conversations.js Removes "no conversations" placeholder message handling
osparc/support/ConversationPage.js Enhances conversation UI with better button layout, ticket ID copying, and conversation renaming
osparc/support/Conversation.js Implements system messages, improved conversation creation flow, and follow-up messaging
osparc/study/Conversation.js Refactors project conversation handling to use event-driven message posting
osparc/store/Support.js Removes deprecated support menu creation function
osparc/navigation/UserMenu.js Adds direct help button to user menu
osparc/navigation/NavigationBar.js Converts help menu to direct help button
osparc/data/model/NodeProgressSequence.js Adds rounded decorator to loading page
osparc/conversation/MessageUI.js Adds support for system messages with special handling
osparc/conversation/AddMessage.js Simplifies message handling by using events instead of direct API calls

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

@odeimaiz odeimaiz marked this pull request as ready for review September 15, 2025 13:50
@codecov
Copy link

codecov bot commented Sep 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.98%. Comparing base (e0f1ec9) to head (82079c8).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8362      +/-   ##
==========================================
+ Coverage   87.87%   87.98%   +0.11%     
==========================================
  Files        1945      586    -1359     
  Lines       75626    25363   -50263     
  Branches     1321        0    -1321     
==========================================
- Hits        66454    22315   -44139     
+ Misses       8774     3048    -5726     
+ Partials      398        0     -398     
Flag Coverage Δ
integrationtests 58.75% <ø> (-5.19%) ⬇️
unittests 87.21% <ø> (+0.66%) ⬆️
Components Coverage Δ
pkg_aws_library ∅ <ø> (∅)
pkg_celery_library ∅ <ø> (∅)
pkg_dask_task_models_library ∅ <ø> (∅)
pkg_models_library ∅ <ø> (∅)
pkg_notifications_library ∅ <ø> (∅)
pkg_postgres_database ∅ <ø> (∅)
pkg_service_integration ∅ <ø> (∅)
pkg_service_library ∅ <ø> (∅)
pkg_settings_library ∅ <ø> (∅)
pkg_simcore_sdk ∅ <ø> (∅)
agent ∅ <ø> (∅)
api_server ∅ <ø> (∅)
autoscaling ∅ <ø> (∅)
catalog ∅ <ø> (∅)
clusters_keeper ∅ <ø> (∅)
dask_sidecar ∅ <ø> (∅)
datcore_adapter ∅ <ø> (∅)
director ∅ <ø> (∅)
director_v2 ∅ <ø> (∅)
dynamic_scheduler ∅ <ø> (∅)
dynamic_sidecar 88.25% <ø> (-2.21%) ⬇️
efs_guardian ∅ <ø> (∅)
invitations ∅ <ø> (∅)
payments ∅ <ø> (∅)
resource_usage_tracker ∅ <ø> (∅)
storage ∅ <ø> (∅)
webclient ∅ <ø> (∅)
webserver 87.94% <ø> (-0.01%) ⬇️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e0f1ec9...82079c8. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@odeimaiz odeimaiz enabled auto-merge (squash) September 15, 2025 14:14
@odeimaiz odeimaiz added the 🤖-automerge marks PR as ready to be merged for Mergify label Sep 15, 2025
@odeimaiz
Copy link
Member Author

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Sep 15, 2025

queue

🟠 Waiting for conditions to match

  • -closed [📌 queue requirement]
  • -conflict [📌 queue requirement]
  • -draft [📌 queue requirement]
  • any of: [📌 queue -> configuration change requirements]
    • -mergify-configuration-changed
    • check-success = Configuration changed
  • any of: [🔀 queue conditions]
    • all of: [📌 queue conditions of queue default]
      • #approved-reviews-by >= 2 [🛡 GitHub branch protection]
      • #approved-reviews-by>=2
      • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
      • #changes-requested-reviews-by=0
      • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
      • #review-threads-unresolved=0
      • -conflict
      • -draft
      • base=master
      • branch-protection-review-decision = APPROVED [🛡 GitHub branch protection]
      • label!=🤖-do-not-merge
      • label=🤖-automerge
      • any of: [🛡 GitHub branch protection]
        • check-skipped = deploy to dockerhub
        • check-neutral = deploy to dockerhub
        • check-success = deploy to dockerhub
      • any of: [🛡 GitHub branch protection]
        • check-success = system-tests
        • check-neutral = system-tests
        • check-skipped = system-tests
      • any of: [🛡 GitHub branch protection]
        • check-success = unit-tests
        • check-neutral = unit-tests
        • check-skipped = unit-tests
      • any of: [🛡 GitHub branch protection]
        • check-success = check OAS' are up to date
        • check-neutral = check OAS' are up to date
        • check-skipped = check OAS' are up to date
      • any of: [🛡 GitHub branch protection]
        • check-success = integration-tests
        • check-neutral = integration-tests
        • check-skipped = integration-tests
      • any of: [🛡 GitHub branch protection]
        • check-success = build-test-images (frontend) / build-test-images
        • check-neutral = build-test-images (frontend) / build-test-images
        • check-skipped = build-test-images (frontend) / build-test-images
      • any of: [🛡 GitHub branch protection]
        • check-success = SonarCloud Code Analysis
        • check-neutral = SonarCloud Code Analysis
        • check-skipped = SonarCloud Code Analysis

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.

This is coming together VERY NICE 🎉

@sonarqubecloud
Copy link

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.

nice!

@odeimaiz odeimaiz merged commit 57d948f into ITISFoundation:master Sep 16, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖-automerge marks PR as ready to be merged for Mergify 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