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

feat: payment processor batch payment native token #1213

Merged
merged 16 commits into from
Oct 27, 2023

Conversation

leoslr
Copy link
Contributor

@leoslr leoslr commented Oct 23, 2023

Description of the changes

Payment-processor supports for batch payment of ETH requests with / without conversion.

Additional change:
Add and use ETH-private instead of using ETH-mainnet on local chain.

@coveralls
Copy link

coveralls commented Oct 24, 2023

Coverage Status

coverage: 87.013% (+0.02%) from 86.994% when pulling 18721ac on feat/payment-processor-batch-payment-native-token into cb21d2a on master.

@olivier7delf
Copy link
Contributor

Great job Leo!
An additional test with ERC and ETH requests paid within a batch could be useful

@leoslr
Copy link
Contributor Author

leoslr commented Oct 26, 2023

Great job Leo! An additional test with ERC and ETH requests paid within a batch could be useful

It's done in this test should pay heterogeneous payments (ETH/ERC20 with/without conversion)

@leoslr leoslr merged commit ca0817f into master Oct 27, 2023
25 of 26 checks passed
@leoslr leoslr deleted the feat/payment-processor-batch-payment-native-token branch October 27, 2023 09:47
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.

4 participants