Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented May 15, 2025

What do these changes do?

  • Show Comp Runs history in Activity Overview

RunsHistory

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz self-assigned this May 15, 2025
@odeimaiz odeimaiz added this to the Bazinga! milestone May 15, 2025
@odeimaiz odeimaiz added t:enhancement Improvement or request on an existing feature a:frontend issue affecting the front-end (area group) labels May 15, 2025
@odeimaiz odeimaiz changed the title 🎨 [Frontend] Show 🎨 [Frontend] Show Runs history May 15, 2025
@odeimaiz odeimaiz marked this pull request as ready for review May 15, 2025 12:31
@odeimaiz odeimaiz requested a review from Copilot May 15, 2025 12:31
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 pull request introduces a history view for computational runs in the Activity Overview.

  • Introduces a new method "fetchJobsHistory" to retrieve runs history.
  • Updates table models and browser components to support configurable run filters (latest vs. historical).
  • Adjusts API endpoint mappings in data resources to reflect the new history and latest active runs endpoints.

Reviewed Changes

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

Show a summary per file
File Description
services/static-webserver/client/source/class/osparc/store/Jobs.js Added "fetchJobsHistory" and updated resource keys/endpoints.
services/static-webserver/client/source/class/osparc/jobs/RunsTableModel.js Updated constructor parameters and conditional fetching of jobs based on history flag.
services/static-webserver/client/source/class/osparc/jobs/RunsTable.js Modified constructor arguments to pass latestOnly and projectUuid.
services/static-webserver/client/source/class/osparc/jobs/RunsBrowser.js Refactored runs-table instantiation to include latestOnly and projectUuid parameters.
services/static-webserver/client/source/class/osparc/jobs/ActivityOverview.js Added layout adjustments and integrated historical runs view.
services/static-webserver/client/source/class/osparc/data/Resources.js Updated endpoints and renamed keys for jobs and subJobs resource fetching.

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

@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.

👍

@odeimaiz odeimaiz added the 🤖-automerge marks PR as ready to be merged for Mergify label May 15, 2025
@odeimaiz
Copy link
Member Author

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented May 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

@odeimaiz odeimaiz requested a review from sanderegg May 15, 2025 14:38
@sonarqubecloud
Copy link

@odeimaiz odeimaiz enabled auto-merge (squash) May 15, 2025 16:07
@odeimaiz odeimaiz merged commit 437adab into ITISFoundation:master May 15, 2025
58 checks passed
@odeimaiz odeimaiz deleted the enh/Comp-Runs-history branch May 15, 2025 16:16
@matusdrobuliak66 matusdrobuliak66 mentioned this pull request Jun 6, 2025
92 tasks
@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) t:enhancement Improvement or request on an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants