Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Jun 16, 2025

What do these changes do?

After the last renamings, the Guided Tours needed to be updated. Also, some new steps were added to them (Templates, Public Projects, Activity Center).

Bonus:

  • Hide Activity Center and Activity Overview for s4llite and tiplite

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz added this to the Engage milestone Jun 16, 2025
@odeimaiz odeimaiz self-assigned this Jun 16, 2025
@odeimaiz odeimaiz added t:enhancement Improvement or request on an existing feature a:frontend issue affecting the front-end (area group) release Preparation for pre-release/release labels Jun 16, 2025
@odeimaiz odeimaiz requested a review from Copilot June 16, 2025 14:31
@odeimaiz odeimaiz marked this pull request as ready for review June 16, 2025 14:32
@odeimaiz odeimaiz added the 🤖-automerge marks PR as ready to be merged for Mergify label Jun 16, 2025
@odeimaiz
Copy link
Member Author

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Jun 16, 2025

queue

🟠 Waiting for conditions to match

  • any of: [🔀 queue conditions]
    • all of: [📌 queue conditions of queue default]
      • #approved-reviews-by >= 2 [🛡 GitHub branch protection]
      • #approved-reviews-by>=2
      • branch-protection-review-decision = APPROVED [🛡 GitHub branch protection]
      • #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
      • 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
  • -closed [📌 queue requirement]
  • -conflict [📌 queue requirement]
  • -draft [📌 queue requirement]
  • any of: [📌 queue -> configuration change requirements]
    • -mergify-configuration-changed
    • check-success = Configuration changed

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

Updates the guided tours to reflect recent renamings (Studies → Projects, Services → Apps), adds new UI tour steps (Activity Center, Templates, Public Projects), and wires up the corresponding UI components.

  • Rename and refresh tour text across all products to use Projects/Templates/Public Projects/Apps.
  • Insert new guided tour steps for Activity Center, Templates, and Public Projects.
  • Add feature-flag logic, IDs, and conditional rendering for the new jobs button and filter items.

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
services/static-webserver/client/source/resource/osparc/tours/tis_tours.json Added Activity Center step; updated notification text.
services/static-webserver/client/source/resource/osparc/tours/tiplite_tours.json Updated filter/search text.
services/static-webserver/client/source/resource/osparc/tours/s4llite_tours.json Updated filter/search text; added Templates and Public Projects steps.
services/static-webserver/client/source/resource/osparc/tours/s4l_tours.json Updated New button and filter text; added Templates, Public Projects, dashboard, and Activity Center steps.
services/static-webserver/client/source/resource/osparc/tours/osparc_tours.json Renamed studies → projects tours; updated dashboard, navigation tours; added Templates, Public Projects, and Activity Center steps.
services/static-webserver/client/source/class/osparc/product/Utils.js Introduced showComputationalActivity helper.
services/static-webserver/client/source/class/osparc/navigation/NavigationBar.js Conditionally render jobs button.
services/static-webserver/client/source/class/osparc/jobs/JobsButton.js Set test ID for jobs button.
services/static-webserver/client/source/class/osparc/dashboard/ResourceDetails.js Conditionally call activity overview popup.
services/static-webserver/client/source/class/osparc/dashboard/ResourceBrowserFilter.js Set test IDs for templates and public projects filter items.

@sonarqubecloud
Copy link

@odeimaiz
Copy link
Member Author

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Jun 16, 2025

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 4d8911d

@mergify mergify bot merged commit 4d8911d into ITISFoundation:master Jun 16, 2025
160 of 163 checks passed
@odeimaiz odeimaiz deleted the enh/update-guided-trours branch June 16, 2025 16:50
@matusdrobuliak66 matusdrobuliak66 mentioned this pull request Aug 5, 2025
88 tasks
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) release Preparation for pre-release/release t:enhancement Improvement or request on an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants