🎨 [Frontend] Runs with children #7712
Conversation
odeimaiz
added
t:enhancement
…imcore into enh/runs-with-children
odeimaiz
requested review from
GitHK,
Copilot,
matusdrobuliak66,
pcrespov and
sanderegg
There was a problem hiding this comment.
Pull Request Overview
This PR updates the frontend to support the new URL parameter “include_children” for improved search and filtering of runs and sub-runs. Key changes include refactoring API endpoints from “jobs” to “runs”, propagating the new includeChildren parameter through various components (data store, table models, and browser components), and updating URL endpoints accordingly.
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| DisabledPlugins.js | Contains a duplicate key definition that may cause conflicts. |
| Jobs.js | Refactors API calls to use “runs” endpoints and propagates includeChildren. |
| SubRunsTableModel.js, SubRunsTable.js, SubRunsBrowser.js, RunsTableModel.js, RunsTable.js, RunsBrowser.js, ActivityOverview.js | Updates constructors and API calls to support the new includeChildren parameter. |
| Resources.js | Updates endpoint URLs to append the include_children parameter as required. |
services/static-webserver/client/source/class/osparc/utils/DisabledPlugins.js
Outdated
Show resolved
Hide resolved
|
|
@Mergifyio queue |
✅ The pull request has been merged automaticallyThe pull request has been merged automatically at 9188df0 |
|
4 participants
What do these changes do?
Following #7718, use the new url parameter
include_childrenRelated issue/s
How to test
Dev-ops