Skip to content

Conversation

@barbaraperic
Copy link
Collaborator

Summary

This PR simplifies the upload progress UI by separating the combined first-stage progress into individual step cards and removing the associated combined progress logic.

Changes

  • Removed combined progress display: Replaced ProgressCardCombined with individual ProgressCard components for each upload step
  • Added CAR creation step: Now displays a dedicated progress card for the "Creating CAR file" step
  • Updated step labels:
    • Combined "creating-car" and "checking-readiness" under the same label "Creating CAR file"
    • Changed "uploading-car" label to "Uploading to Filecoin SP"
  • Removed estimated times: Eliminated time estimates for creating-car, checking-readiness, and uploading-car steps
  • Code cleanup: Deleted unused components and hooks:
    • progress-card-combined.tsx
    • use-first-stage-state.ts
    • getFirstStageProgress function from upload-progress.ts

@barbaraperic barbaraperic self-assigned this Nov 6, 2025
@FilOzzy FilOzzy added team/filecoin-pin "Filecoin Pin" project is a stakeholder for this work. team/fs-wg FOC working group is a stakeholder for this work, and thus wants to track it on their project board. labels Nov 6, 2025
@FilOzzy FilOzzy added this to FS Nov 6, 2025
@github-project-automation github-project-automation bot moved this to 📌 Triage in FS Nov 6, 2025
@vercel
Copy link

vercel bot commented Nov 6, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
filecoin-pin-website Ready Ready Preview Comment Nov 6, 2025 1:17pm

@github-actions
Copy link

github-actions bot commented Nov 6, 2025

Filecoin Pin Upload

IPFS Artifacts:

Onchain verification:

Payment:

  • Current Filecoin Pay balance: 49.0116 USDFC
  • Amount deposited to Filecoin Pay by this workflow: 0.0000 USDFC
  • Data Set Storage runway: 4 year(s) 1 month(s) 11 day(s)

More details

@BigLep BigLep moved this from 📌 Triage to ⌨️ In Progress in FS Nov 7, 2025
@BigLep
Copy link
Member

BigLep commented Nov 7, 2025

@barbaraperic : I assume you'll mark this ready for review and assign to someone like Russell when you're ready.

@barbaraperic barbaraperic marked this pull request as ready for review November 10, 2025 11:56
Copy link
Collaborator

@SgtPooki SgtPooki left a comment

Choose a reason for hiding this comment

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

definitely cleaned up some code with this one. i think this is a good call for UX and maintainers.

@github-project-automation github-project-automation bot moved this from ⌨️ In Progress to ✔️ Approved by reviewer in FS Nov 10, 2025
@barbaraperic barbaraperic merged commit 6cbdefe into main Nov 10, 2025
4 checks passed
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FS Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team/filecoin-pin "Filecoin Pin" project is a stakeholder for this work. team/fs-wg FOC working group is a stakeholder for this work, and thus wants to track it on their project board.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants