forked from Cloud-Pipelines/pipeline-editor
-
Notifications
You must be signed in to change notification settings - Fork 5
Cleanup ExecutionDetails #1471
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-04-add_ui_blocks_to_context_panel
Choose a base branch
from
12-04-cleanup_executiondetails
base: 12-04-add_ui_blocks_to_context_panel
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 ExecutionDetails #1471
camielvs
wants to merge
1
commit into
12-04-add_ui_blocks_to_context_panel
from
12-04-cleanup_executiondetails
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
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. |
54e3528 to
e6ea5d1
Compare
49d4edd to
533c465
Compare
e6ea5d1 to
87a5600
Compare
533c465 to
81d703c
Compare
87a5600 to
8f16f7a
Compare
a81e889 to
86d3812
Compare
a526425 to
44309e3
Compare
86d3812 to
2c21828
Compare
44309e3 to
d385e99
Compare
869f4cf to
6dd07d1
Compare
d385e99 to
66a011a
Compare
db9d2a2 to
8bc2c30
Compare
17b4f45 to
f57fb5d
Compare
8bc2c30 to
1728b26
Compare
f57fb5d to
1d45169
Compare
aa47905 to
8a01b0d
Compare
1d45169 to
1cb8f58
Compare
3 tasks
3 tasks
1cb8f58 to
4814e7c
Compare
10df8cb to
113b8c5
Compare
4821940 to
73b3f03
Compare
16212fb to
ddf0ce3
Compare
73b3f03 to
8172705
Compare
ddf0ce3 to
ef66839
Compare
8172705 to
19fee5f
Compare
3 tasks
19fee5f to
7f0ee6f
Compare
ef66839 to
ad96688
Compare
7f0ee6f to
f3260f1
Compare
ad96688 to
c8b7768
Compare
3 tasks
f3260f1 to
436b867
Compare
c8b7768 to
00ae3d5
Compare
436b867 to
d89ccdb
Compare
00ae3d5 to
bde6c26
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
Moves
ExecutionDetailsonto UI Primitives and the new Context Panel Blocks.Also colocates the file with TaskDetails, since the component is not being consumed anywhere else.
Related Issue and Pull requests
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