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

fix: missing eth-proxy on BSC #791

Merged
merged 2 commits into from
Mar 1, 2022
Merged

fix: missing eth-proxy on BSC #791

merged 2 commits into from
Mar 1, 2022

Conversation

benjlevesque
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented Feb 28, 2022

Coverage Status

Coverage remained the same at 89.371% when pulling 6b70dfb on fix-eth-proxy-bsc into ee3436e on master.

@benjlevesque benjlevesque enabled auto-merge (squash) March 1, 2022 10:27
@benjlevesque benjlevesque merged commit b1a73c0 into master Mar 1, 2022
@benjlevesque benjlevesque deleted the fix-eth-proxy-bsc branch March 1, 2022 10:34
olivier7delf pushed a commit that referenced this pull request Mar 2, 2022
olivier7delf pushed a commit that referenced this pull request Mar 2, 2022
olivier7delf pushed a commit that referenced this pull request Mar 2, 2022
olivier7delf pushed a commit that referenced this pull request Mar 2, 2022
olivier7delf pushed a commit that referenced this pull request Mar 2, 2022
olivier7delf pushed a commit that referenced this pull request Mar 2, 2022
olivier7delf pushed a commit that referenced this pull request Mar 2, 2022
Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>
olivier7delf added a commit that referenced this pull request Mar 2, 2022
* modification to allow payment with wrong fee address, test for erc20 fee-proxy-contract.test

Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

* feat: ether to smart wallet payment (#780)

Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

* fix: missing eth-proxy on BSC (#791)

Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

* feat: Start of SuperFluid integration (#671)

Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

* test ok on any-to-erc20-proxy-contract.test.ts

Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

* clear comment

Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

* clean comments

Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

* Update any-to-erc20-proxy-contract.test.ts

rename the test, to understand that:
- payment & refund work even with the wrong feeAddress
- feeBalance sum only if feeAddress is fine

Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

* Update any-to-erc20-proxy-contract.test.ts

clean comments

Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

* Update fee-proxy-contract.test.ts

rename the test, to underline that:
- payment & refund work even with the wrong feeAddress
- feeBalance sum only if feeAddress is fine

Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

* Update packages/payment-detection/test/erc20/fee-proxy-contract.test.ts

Co-authored-by: Alexandre ABRIOUX <alexandre-abrioux@users.noreply.github.com>
Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

* Update packages/payment-detection/test/erc20/fee-proxy-contract.test.ts

Co-authored-by: Alexandre ABRIOUX <alexandre-abrioux@users.noreply.github.com>
Signed-off-by: olivier7delf <55892112+olivier7delf@users.noreply.github.com>

Co-authored-by: leoslr <50319677+leoslr@users.noreply.github.com>
Co-authored-by: Benjamin Levesque <14175665+benjlevesque@users.noreply.github.com>
Co-authored-by: Bertrand Juglas <b@juglas.name>
Co-authored-by: Alexandre ABRIOUX <alexandre-abrioux@users.noreply.github.com>
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