forked from Cloud-Pipelines/pipeline-editor
-
Notifications
You must be signed in to change notification settings - Fork 4
Cleanup RunDetails #1508
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
camielvs
wants to merge
1
commit into
12-03-cleanup_pipelinedetails
Choose a base branch
from
12-09-cleanup_rundetails
base: 12-03-cleanup_pipelinedetails
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Cleanup RunDetails #1508
+81
−223
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Dec 9, 2025
Collaborator
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
357c105 to
4f1da94
Compare
8a732e2 to
043bd18
Compare
4f1da94 to
c9f9463
Compare
b4df554 to
68f0eb0
Compare
c9f9463 to
bf46c51
Compare
12baeb2 to
7f0da59
Compare
bf46c51 to
2ef9e9e
Compare
7f0da59 to
15bad0b
Compare
2ef9e9e to
3464c13
Compare
15bad0b to
2573d44
Compare
500ca6f to
c135e7f
Compare
2573d44 to
73e7131
Compare
This was referenced Dec 10, 2025
c135e7f to
d0edb0b
Compare
2a0ccca to
f8e3134
Compare
d0edb0b to
887c783
Compare
e677909 to
65234bf
Compare
4d838bd to
cdd5672
Compare
65234bf to
7617cac
Compare
16e34dc to
d131b35
Compare
7617cac to
c74095b
Compare
d131b35 to
aa5a32f
Compare
c74095b to
5c47b5a
Compare
3 tasks
5c47b5a to
1e9e0cb
Compare
aa5a32f to
05071b0
Compare
1e9e0cb to
0550c7e
Compare
05071b0 to
1d06828
Compare
3 tasks
1d06828 to
73ed3da
Compare
0550c7e to
2fc0aeb
Compare
73ed3da to
4ba0f1d
Compare
2fc0aeb to
2c5eafa
Compare
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.

Description
RunDetails recently got a good cleanup, so this PR focusses mostly on bringing it over to the ContextPanel Block system for improved visual consistency.
Functionality of the ArtifactsList is already moved to PipelineIO, so this last implementation of it has been removed along with the component itself.
Related Issue and Pull requests
Closes https://github.com/Shopify/oasis-frontend/issues/401
Type of Change
Checklist
Screenshots (if applicable)
Before:
After:
Test Instructions
No change to app functionality. UI update only. Confirm that the interface works and shows info as expected.
Additional Comments