Skip to content
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

usdc recurring donation decimals are incorrect and transactions reverts #4861

Open
divine-comedian opened this issue Nov 10, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@divine-comedian
Copy link
Contributor

when adding a recurring donation in usdc it does not seem to parse the decimals correctly.

It does not estimate the amount of months left from the proposed amount to deposit and amount to stream monthly - when going to the confirmaiton modal it shows an amount MUCH lower than the expected, based on the amounts input

in addition it appears the optional donation to giveth toggle is not working correctly - it always applies a donation to giveth even when you toggle it off!!

Screen.Recording.2024-11-10.074357.mp4
@divine-comedian divine-comedian added the bug Something isn't working label Nov 10, 2024
@github-project-automation github-project-automation bot moved this to New Issues in All-Devs Nov 10, 2024
@divine-comedian divine-comedian moved this from New Issues to Dev Research in All-Devs Nov 10, 2024
@kkatusic
Copy link
Collaborator

@divine-comedian done hotfix for this USDC token, problem was with it's decimal, but also found problem related with months number, it wasn't ok. Other problem that you mentioned like applying donation to Giveth if you unchecked it didn't saw and also in video that you sent doesn't showing.

@kkatusic kkatusic moved this from Dev Research to Code Review/PR in All-Devs Nov 11, 2024
@divine-comedian
Copy link
Contributor Author

@kkatusic from our discussion yesterday it appears that there is still an error in the flow somewhere

image

Things we've tested

  • make recurring donation with other ERC20 tokens (OP & GIV) - WORKS ✅
  • wrap USDC on superfluid app, make recurring donation with already wrapped USDC - WORKS ✅
  • make recurring donation with ETH - WORKS ✅
  • double checked configs for correct token addresses - WORKS ✅
    -deposit tokens through my recurring donation page - FAILS ❌ - it seems to revoke the approve instead of approve the allowance to deposit the token

image

It seems this could be related to how we handle depositing tokens or how we batch the deposit function together with creating a stream... bearing in mind USDC is the only token that has 6 decimal precision while all other tokens have 18.. the super USDC token has 18 decimals

@divine-comedian divine-comedian changed the title usdc recurring donation decimals are incorrect and option donation toggle doesn't work usdc recurring donation decimals are incorrect and transactions reverts Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Code Review/PR
Development

No branches or pull requests

2 participants