Skip to content

Fix typescript issues in repo#327

Open
SwissBitcoinPay wants to merge 2 commits intomainfrom
cursor/fix-typescript-issues-in-repo-df76
Open

Fix typescript issues in repo#327
SwissBitcoinPay wants to merge 2 commits intomainfrom
cursor/fix-typescript-issues-in-repo-df76

Conversation

@SwissBitcoinPay
Copy link
Owner

Fix a significant number of TypeScript issues to improve type safety and reduce runtime errors.

This PR addresses 116 TypeScript errors across 20 files, reducing the total error count from 447 to 331. Key fixes include adding the id property to StringPart objects, resolving parameter type mismatches in various utility functions, improving the wallet type system by exporting missing types and correcting function signatures, and fixing hardware wallet integration issues related to Ledger and Trezor transaction signing.


Open in Cursor Open in Web

This commit refactors wallet-related types and functions, improving type safety and code organization. It also includes minor updates to utility functions for better consistency.

Co-authored-by: authenticator.cursor.tubeless137 <authenticator.cursor.tubeless137@passmail.net>
@cursor
Copy link

cursor bot commented Sep 9, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@netlify
Copy link

netlify bot commented Sep 9, 2025

Deploy Preview for app-swiss-bitcoin-pay-ch ready!

Name Link
🔨 Latest commit ca77429
🔍 Latest deploy log https://app.netlify.com/projects/app-swiss-bitcoin-pay-ch/deploys/68c0571ba0c150000819fec1
😎 Deploy Preview https://deploy-preview-327--app-swiss-bitcoin-pay-ch.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Sep 9, 2025

Deploy Preview for swissbitcoinpayapp ready!

Name Link
🔨 Latest commit ca77429
🔍 Latest deploy log https://app.netlify.com/projects/swissbitcoinpayapp/deploys/68c0571b232f2300089e004a
😎 Deploy Preview https://deploy-preview-327--swissbitcoinpayapp.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@SwissBitcoinPay SwissBitcoinPay marked this pull request as ready for review September 9, 2025 13:00
Add View wrappers to various components for better structure. Update type assertions and fix minor issues in Bitbox API and SendModal.

Co-authored-by: authenticator.cursor.tubeless137 <authenticator.cursor.tubeless137@passmail.net>
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.

2 participants