-
Notifications
You must be signed in to change notification settings - Fork 242
feat: add typescript/token-tax-calculator example (DeFi tax estimator) #129
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
base: main
Are you sure you want to change the base?
Conversation
|
Thanks for adding this example — it’s a nice lightweight educational demo for compliance/accounting use cases 👍. One suggestion: right now the flow relies only on self-input for transactions, but it would feel more practical and compelling if there were at least a minimal on-chain interaction. For example, you could: Add a “Connect Wallet” button (via MetaMask or similar) to fetch the user’s address. Use a free API (e.g., Etherscan, Covalent, Alchemy Transfers API) to pull recent ERC-20 / swap transactions. Run the existing tax calculation logic against those fetched transactions, so users can see how the estimator works without manually typing everything in. This would keep the example simple, but demonstrate how a frontend-only tool can still interact with on-chain data in a compliance context. |
|
Thank you for your contribution. Please provide a complete runnable code, which can be generated using create-next-app or CRA. It would be even better if you could deploy it to Vercel for online review. @flashtxh |
futures996
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.
ESSE COMITE ESTÁ APROVADO, VAMOS
futures996
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.
OK
|
Hvala vam na vašem doprinosu. Molimo vas da dostavite kompletan kod koji se može generirati pomoću create-next-app ili CRA. Bilo bi još bolje kada biste ga mogli implementirati u Vercel za online pregled |
|
OK |
0xlucasliao
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.
Any updates on the Chain Integration and Runnable code?
|
I believe there is nothing new after the changes made to the repository! Let's go to BNB SMART CHAIN |
0xlucasliao
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.
A bump on the feedback previously mentioned:
right now the flow relies only on self-input for transactions, but it would feel more practical and compelling if there were at least a minimal on-chain interaction. For example, you could:
Add a “Connect Wallet” button (via MetaMask or similar) to fetch the user’s address.
Use a free API (e.g., Etherscan, Covalent, Alchemy Transfers API) to pull recent ERC-20 / swap transactions.
Run the existing tax calculation logic against those fetched transactions, so users can see how the estimator works without manually typing everything in.
This would keep the example simple, but demonstrate how a frontend-only tool can still interact with on-chain data in a compliance context.
What’s new
typescript/token-tax-calculatorexample (educational token swap tax estimator).web/list.jsonwith a new catalog entry and guide link.Why
How to review
typescript/token-tax-calculator/README.mdand confirm content.web/list.jsoncontains the new card with correct title, description, tags, github, and guide.