-
Notifications
You must be signed in to change notification settings - Fork 13
Fix/upload credits #37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
bhavyagor12
commented
Dec 11, 2025
- wallets with insufficient balance get errors throw on failed upload
- 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
|
🚀 Snapshot release published: Install with: pnpm add permaweb-deploy@0.0.0-alpha-20251211100922 |
dtfiedler
left a comment
There was a problem hiding this 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
|
@dtfiedler thanks for the review!! I ran - Let me know if I m doing something wrong, thanks - Screen.Recording.2025-12-12.at.12.52.08.PM.mov |
|
🚀 Snapshot release published: Install with: pnpm add permaweb-deploy@0.0.0-alpha-20251218112416 |
|
🚀 Snapshot release published: Install with: pnpm add permaweb-deploy@0.0.0-alpha-20251218112422 |
|
🚀 Snapshot release published: Install with: pnpm add permaweb-deploy@0.0.0-alpha-20251218112729 |
|
🚀 Snapshot release published: Install with: pnpm add permaweb-deploy@0.0.0-alpha-20260107222737 |