Incompatibility with USDT (Non-Standard ERC20s):

[aniket866]

• Issue: The contract uses IERC20 which expects transfer and transferFrom to return a bool . USDT and other older tokens return void.

• Impact: Payments with USDT (and similar tokens) will always revert, making the contract unusable for the most popular stablecoin.

@kumawatkaran523 Please assign

[sach2004]

@kumawatkaran523 @DengreSarthak it is valid. USDT transferFrom returns nothing instead of bool so the abi decode will revert every time. invoices with USDT as payment token can be created but never paid, pretty important since USDT is on all the chains this contract is deployed on.

[Anand-0037]

Hi @Atharva0506, I reproduced this on the latest main using a mock token whose transferFrom() returns no value. Both single and batch payments revert and roll back.
I noticed that @aniket866 had already asked to work on this. I also checked PRs #201 and #203; they touch related tests and batch-payment code but do not appear to handle no-return tokens.
Is anyone currently working on #86? If not, I would like to take it. I plan to cover no-return, true-return, and false-return tokens for both payment paths and then implement the approach you prefer.