Skip to content

fix: v2: order pipeline list by ModifiedAt#2263

Merged
camielvs merged 1 commit into
masterfrom
05-13-fix_order_pipeline_list_by_modifiedat
May 14, 2026
Merged

fix: v2: order pipeline list by ModifiedAt#2263
camielvs merged 1 commit into
masterfrom
05-13-fix_order_pipeline_list_by_modifiedat

Conversation

@camielvs
Copy link
Copy Markdown
Collaborator

@camielvs camielvs commented May 13, 2026

Description

The open pipeline dialog was ordering pipelines somewhat randomly. now they are order by their last modified date, as per the pipeline dashboard

Related Issue and Pull requests

Type of Change

  • Improvement

Checklist

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

Screenshots (if applicable)

image.png

Test Instructions

Additional Comments

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 13, 2026

🎩 Preview

A preview build has been created at: 05-13-fix_order_pipeline_list_by_modifiedat/c43a6fe

@camielvs camielvs changed the title fix: order pipeline list by ModifiedAt fix: v2: order pipeline list by ModifiedAt May 13, 2026
@camielvs camielvs marked this pull request as ready for review May 14, 2026 00:48
@camielvs camielvs requested a review from a team as a code owner May 14, 2026 00:48
Comment on lines +106 to +109
.sort(
(a, b) =>
(b.modifiedAt?.getTime() ?? 0) - (a.modifiedAt?.getTime() ?? 0),
);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Future: We may think to add custom order by - click on column to select asc/desc and column

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

agreed. smart tables

@camielvs camielvs force-pushed the 05-13-fix_order_pipeline_list_by_modifiedat branch from ac7738d to 9f3564f Compare May 14, 2026 23:30
@camielvs camielvs force-pushed the 05-13-fix_v2_debug_panel_yaml_viewer_has_no_height branch from 21f3802 to 8469e82 Compare May 14, 2026 23:30
@camielvs camielvs changed the base branch from 05-13-fix_v2_debug_panel_yaml_viewer_has_no_height to graphite-base/2263 May 14, 2026 23:33
@camielvs camielvs force-pushed the 05-13-fix_order_pipeline_list_by_modifiedat branch from 9f3564f to 26b534a Compare May 14, 2026 23:33
@camielvs camielvs force-pushed the graphite-base/2263 branch from 8469e82 to c6c8785 Compare May 14, 2026 23:33
@graphite-app graphite-app Bot changed the base branch from graphite-base/2263 to master May 14, 2026 23:34
@camielvs camielvs force-pushed the 05-13-fix_order_pipeline_list_by_modifiedat branch from 26b534a to c43a6fe Compare May 14, 2026 23:34
Copy link
Copy Markdown
Collaborator Author

camielvs commented May 14, 2026

Merge activity

  • May 14, 11:37 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 14, 11:37 PM UTC: @camielvs merged this pull request with Graphite.

@camielvs camielvs merged commit 1b6ba4f into master May 14, 2026
17 checks passed
@camielvs camielvs deleted the 05-13-fix_order_pipeline_list_by_modifiedat branch May 14, 2026 23:37
@camielvs camielvs mentioned this pull request May 15, 2026
3 tasks
@camielvs camielvs added the #gsd:50471 CSOM-X label May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants