Skip to content

Conversation

@Mbeaulne
Copy link
Collaborator

@Mbeaulne Mbeaulne commented Dec 5, 2025

Description

Created a reusable ArtifactsList component to standardize the display of pipeline inputs and outputs across the application. This component provides a consistent UI for showing artifact details with optional actions like copying values and editing.

Type of Change

  • Improvement
  • Cleanup/Refactor

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

Screenshot 2025-12-05 at 1.52.15 PM.png

Screenshot 2025-12-05 at 1.52.32 PM.png

Test Instructions

  1. Navigate to the Pipeline Editor and verify that inputs and outputs are displayed correctly in the Pipeline Details panel
  2. Check that the edit functionality for inputs and outputs still works as expected
  3. View a Pipeline Run and confirm that the artifacts are displayed properly

Additional Comments

The new component improves code maintainability by eliminating duplicate markup and standardizing the artifact display pattern across different parts of the application.

Copy link
Collaborator Author

Mbeaulne commented Dec 5, 2025

@Mbeaulne Mbeaulne mentioned this pull request Dec 5, 2025
8 tasks
@Mbeaulne Mbeaulne marked this pull request as ready for review December 5, 2025 18:06
@Mbeaulne Mbeaulne force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch from 8e09035 to d5fc138 Compare December 5, 2025 18:45
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from 0fa7317 to 9e3cb25 Compare December 5, 2025 18:45
@Mbeaulne Mbeaulne force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch from d5fc138 to 4b4bf4c Compare December 5, 2025 19:15
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from 9e3cb25 to 5c08509 Compare December 5, 2025 19:15
@Mbeaulne Mbeaulne force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch from 4b4bf4c to 20cb3bc Compare December 5, 2025 19:25
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch 2 times, most recently from 98090bc to 53b5562 Compare December 5, 2025 19:56
@Mbeaulne Mbeaulne mentioned this pull request Dec 5, 2025
8 tasks
@Mbeaulne Mbeaulne force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch from 20cb3bc to 921cf8a Compare December 5, 2025 20:03
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch 2 times, most recently from f57e566 to 3a42be6 Compare December 5, 2025 20:07
@Mbeaulne Mbeaulne force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch from 921cf8a to 044bf4c Compare December 5, 2025 20:14
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from 3a42be6 to 1c2dbbd Compare December 5, 2025 20:14
@Mbeaulne Mbeaulne force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch from 044bf4c to 2d1c29d Compare December 5, 2025 21:25
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from 1c2dbbd to d536aa2 Compare December 5, 2025 21:25
@Mbeaulne Mbeaulne force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch from 2d1c29d to 062519d Compare December 5, 2025 21:29
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from d536aa2 to af1ece0 Compare December 5, 2025 21:29
Copy link
Collaborator

@camielvs camielvs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This overlaps with the work I've done in #1468 but generally seems good. We can merge this in and then I will adapt what I have to see what the differences are.

@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from af1ece0 to 554bf84 Compare December 5, 2025 21:47
@Mbeaulne Mbeaulne force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch from 062519d to 55c2bc6 Compare December 5, 2025 22:00
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from 554bf84 to 0f80e24 Compare December 5, 2025 22:00
@Mbeaulne Mbeaulne force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch from 55c2bc6 to ec2d7b7 Compare December 6, 2025 15:15
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from 0f80e24 to 64b1c54 Compare December 6, 2025 15:15
@camielvs camielvs force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch from ec2d7b7 to e370733 Compare December 6, 2025 15:17
@camielvs camielvs force-pushed the 12-05-abstracts_artifacts_list branch from 64b1c54 to fd7d491 Compare December 6, 2025 15:18
@Mbeaulne Mbeaulne force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch from e370733 to 11c3d4b Compare December 6, 2025 15:21
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from fd7d491 to 79d5e3e Compare December 6, 2025 15:21
@Mbeaulne Mbeaulne force-pushed the 12-05-cleans_up_pipelinedetails_v2 branch 2 times, most recently from 7205755 to 5fd9355 Compare December 6, 2025 15:27
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from 79d5e3e to 86ec640 Compare December 6, 2025 15:27
@Mbeaulne Mbeaulne changed the base branch from 12-05-cleans_up_pipelinedetails_v2 to graphite-base/1481 December 6, 2025 15:30
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from 86ec640 to 4d8f0ba Compare December 6, 2025 15:30
@graphite-app graphite-app bot changed the base branch from graphite-base/1481 to master December 6, 2025 15:30
@Mbeaulne Mbeaulne force-pushed the 12-05-abstracts_artifacts_list branch from 4d8f0ba to e34cd62 Compare December 6, 2025 15:30
@Mbeaulne Mbeaulne merged commit 6b33171 into master Dec 6, 2025
5 checks passed
Copy link
Collaborator Author

Mbeaulne commented Dec 6, 2025

Merge activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants