Feature #13327
closedMore responsive page loads with API ?include=container_uuid
Description
These all need to be modified to use the group contents API, as that's the only one that supports ?include=container_uuid
. This shouldn't be a problem since you can easily query container requests through the contents API.
- Project listing
- Other process listings (all processes, subprocesses, workflow definition history)
- Loading the process panel details (currently the status badge often flickers because there's a delay in getting the correct status)
Related issues
Updated by Tom Morris over 6 years ago
- Related to Feature #12917: Support ?include=container_uuid for group contents added
Updated by Tom Morris over 6 years ago
- Subject changed from Use new container_request?include=container to [Workbench] Use new API container_request?include=container
Updated by Peter Amstutz about 5 years ago
- Related to Bug #15672: List of subprocesses limited in process view added
Updated by Peter Amstutz almost 5 years ago
- Target version set to 2020-02-12 Sprint
Updated by Peter Amstutz almost 5 years ago
- Category set to Workbench2
- Subject changed from [Workbench] Use new API container_request?include=container to Use new API container_request?include=container
Updated by Lucas Di Pentima almost 5 years ago
- Assigned To set to Lucas Di Pentima
Updated by Lucas Di Pentima almost 5 years ago
- Related to deleted (Feature #12917: Support ?include=container_uuid for group contents)
Updated by Lucas Di Pentima almost 5 years ago
- Blocked by Feature #12917: Support ?include=container_uuid for group contents added
Updated by Lucas Di Pentima over 4 years ago
- Target version changed from 2020-02-12 Sprint to 2020-02-26 Sprint
Updated by Lucas Di Pentima over 4 years ago
- Target version deleted (
2020-02-26 Sprint) - Assigned To deleted (
Lucas Di Pentima)
Updated by Peter Amstutz over 1 year ago
- Subject changed from Use new API container_request?include=container to More responsive page loads with API ?include=container_uuid
Updated by Peter Amstutz over 1 year ago
- Release deleted (
60) - Target version set to Future
Updated by Peter Amstutz 5 months ago
- Target version changed from Future to Development 2024-07-24 sprint
Updated by Peter Amstutz 5 months ago
- Target version changed from Development 2024-07-24 sprint to 439
Updated by Peter Amstutz 5 months ago
- Target version changed from 439 to Development 2024-06-19 sprint
Updated by Peter Amstutz 5 months ago
- Target version changed from Development 2024-06-19 sprint to Development 2024-07-03 sprint
Updated by Peter Amstutz 5 months ago
- Target version changed from Development 2024-07-03 sprint to Development 2024-07-24 sprint
Updated by Peter Amstutz 4 months ago
- Target version changed from Development 2024-07-24 sprint to Development 2024-08-07 sprint
Updated by Peter Amstutz 4 months ago
- Target version changed from Development 2024-08-07 sprint to Development 2024-08-28 sprint
Updated by Peter Amstutz 2 months ago
13327-include-container-uuid @ 5ae2812a591fe7335599d5202f9bd4e4090ebd60
- All agreed upon points are implemented / addressed.
- search, project page, process page, and workflow page should all be using the new API
- Anything not implemented (discovered or discussed during work) has a follow-up story.
- n/a
- Code is tested and passing, both automated and manual, what manual testing was done is described
- waiting to see if it passes existing integration tests
- need to add a test for the API fix for using 'container' filter on container requests with group contents
- Documentation has been updated.
- n/a
- Behaves appropriately at the intended scale (describe intended scale).
- should improve loading time by collapsing two requests into one, as well as solving a longstanding issue where containers show the wrong status
- Considered backwards and forwards compatibility issues between client and server.
- the search panel is the one place where you're likely to see API mismatch between client and server, so the search middleware checks the API revision and determines which API to use
- the other places do not, on the assumption that workbench 2 is upgraded together with its "home" API server.
- Follows our coding standards and GUI style guidelines.
- yes
Updated by Peter Amstutz 2 months ago
- Target version changed from Development 2024-08-28 sprint to Development 2024-09-11 sprint
Updated by Peter Amstutz about 2 months ago
- Target version changed from Development 2024-09-11 sprint to Development 2024-09-25 sprint
Updated by Peter Amstutz about 2 months ago
13327-include-container-uuid @ dd2ded8399c52a331ecdd300da2c270061c44c9a
- added a test case in API server groups_controller_test for fix I mentioned in the previous comment.
- updated the API intercepts to reflect they are using group contents instead of the container_request/container
- passes tests
Updated by Lisa Knox about 2 months ago
- /processes-actions.ts L55: there is nothing in the catch block, meaning all exceptions will be ignored
- /search-results-middleware-service.ts L78, L100, L141, L158: I always twinge when I see a hardcoded value like this, even though we know it will never change. I recommend replacing it with a string constant at the top of the file (or maybe export it as a constant from /common/app-info.ts along with any other important version numbers, e.g. search-bar-actions.ts L381)
- /processes-middleware-service.ts L95-96: remove commented out code
- several unused imports/variables warnings in the terminal when running workbench, a few unrelated to this ticket
Updated by Peter Amstutz about 2 months ago
Lisa Knox wrote in #note-35:
Very thorough, just a few cleanup notes:
- /processes-actions.ts L55: there is nothing in the catch block, meaning all exceptions will be ignored
I added a snackbar error.
- /search-results-middleware-service.ts L78, L100, L141, L158: I always twinge when I see a hardcoded value like this, even though we know it will never change. I recommend replacing it with a string constant at the top of the file (or maybe export it as a constant from /common/app-info.ts along with any other important version numbers, e.g. search-bar-actions.ts L381)
Good call, they are symbolic constants in app-info.ts now.
- /processes-middleware-service.ts L95-96: remove commented out code
Done.
- several unused imports/variables warnings in the terminal when running workbench, a few unrelated to this ticket
I fixed all the warnings I saw.
13327-include-container-uuid @ cf84bf3fe656d25e9a081f4e3392375ec8b63f5a
Updated by Peter Amstutz about 2 months ago
There were some conflicts, so I rebased on main:
13327-include-container-uuid @ c18a6647f4431516e687e3a226d5be23854ddc89
Updated by Peter Amstutz about 1 month ago
13327-include-container-uuid @ 31eb5c0d41ef8ab362e93af23df3991aa433dea4
Updated by Peter Amstutz about 1 month ago
- Status changed from In Progress to Resolved