Skip to content

feat: Add action summary to push view#1446

Open
andypols wants to merge 5 commits intofinos:mainfrom
qube-rt:display-git-push-actions
Open

feat: Add action summary to push view#1446
andypols wants to merge 5 commits intofinos:mainfrom
qube-rt:display-git-push-actions

Conversation

@andypols
Copy link
Contributor

@andypols andypols commented Mar 5, 2026

Summary

It can be hard to understand why a push was rejected. This PR adds a view of each step involved in a push, making it easier to identify failures. It also fixes #1387 and prevents the UI from crashing when a push fails before a diff is generated.

We use "steps" instead of actions in the UI, since “actions” was confusing for users.

Solution

  • Introduces a tabbed view for push data, including Commits, Diff, and Steps. This reduces scrolling and provides a foundation for future additions, such as an AI-generated change summary (future PR).
CleanShot 2026-03-06 at 08 38 11@2x
  • Adds dedicated components for each tab.
  • Extended CustomTabs to support an optional badge.
  • Calculates the total error count across steps and displays it as a badge to make failures more visible.
  • Sums the error count from the steps and displays an error count as a badge to highlight there are errors.
  • Adds a StepsTimeline component featuring an accordion layout, status chips, and log output to better surface errors, blocked steps, and successful execution.
CleanShot 2026-03-06 at 08 39 41@2x
  • Clicking on an accordion step the info about that action
CleanShot 2026-03-06 at 08 42 19@2x
  • Prevents large step payloads from being rendered.

@andypols andypols requested a review from a team as a code owner March 5, 2026 23:36
@netlify
Copy link

netlify bot commented Mar 5, 2026

Deploy Preview for endearing-brigadeiros-63f9d0 canceled.

Name Link
🔨 Latest commit 9e287a1
🔍 Latest deploy log https://app.netlify.com/projects/endearing-brigadeiros-63f9d0/deploys/69aa8ef6593dd00008711a5d

@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.01%. Comparing base (ac353de) to head (9e287a1).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1446   +/-   ##
=======================================
  Coverage   90.01%   90.01%           
=======================================
  Files          67       67           
  Lines        4769     4769           
  Branches      885      885           
=======================================
  Hits         4293     4293           
  Misses        458      458           
  Partials       18       18           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

Pushes in the 'error' category don't render in the admin UI unless a diff was produced

1 participant