Skip to content

Conversation

@bhavyagor12
Copy link
Member

  1. wallets with insufficient balance get errors throw on failed upload
  2. check upload costs before upload folder

mid changes with checking upload credits
1. focus on throwing error on insufficient funds
2. added mocks for wallets without balance
3. throw error on missing txOrManifestId
@bhavyagor12 bhavyagor12 requested a review from NickJ202 December 11, 2025 10:08
@github-actions
Copy link

🚀 Snapshot release published: permaweb-deploy@0.0.0-alpha-20251211100922

Install with:

pnpm add permaweb-deploy@0.0.0-alpha-20251211100922

@dtfiedler dtfiedler self-requested a review December 11, 2025 19:55
Copy link
Collaborator

@dtfiedler dtfiedler left a comment

Choose a reason for hiding this comment

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

@bhavyagor12 - please run prettier

@bhavyagor12
Copy link
Member Author

@dtfiedler thanks for the review!!

I ran - pnpm run format
No changes are recognized by git / files remain unchanged

Let me know if I m doing something wrong, thanks -

Screen.Recording.2025-12-12.at.12.52.08.PM.mov

@github-actions
Copy link

🚀 Snapshot release published: permaweb-deploy@0.0.0-alpha-20251218112416

Install with:

pnpm add permaweb-deploy@0.0.0-alpha-20251218112416

@github-actions
Copy link

🚀 Snapshot release published: permaweb-deploy@0.0.0-alpha-20251218112422

Install with:

pnpm add permaweb-deploy@0.0.0-alpha-20251218112422

@github-actions
Copy link

🚀 Snapshot release published: permaweb-deploy@0.0.0-alpha-20251218112729

Install with:

pnpm add permaweb-deploy@0.0.0-alpha-20251218112729

@NickJ202 NickJ202 self-assigned this Jan 7, 2026
@NickJ202 NickJ202 removed their assignment Jan 7, 2026
@github-actions
Copy link

github-actions bot commented Jan 7, 2026

🚀 Snapshot release published: permaweb-deploy@0.0.0-alpha-20260107222737

Install with:

pnpm add permaweb-deploy@0.0.0-alpha-20260107222737

@NickJ202 NickJ202 merged commit 510a008 into alpha Jan 7, 2026
5 checks passed
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.

4 participants