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.

Sign up for GitHub

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

Reduce ERC20 allowance before triggering transfer #3056

Merged
merged 11 commits into from
Dec 31, 2021
Merged

Reduce ERC20 allowance before triggering transfer #3056

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
4309312
Reduce ERC20 allowance before triggering transfer
frangio Dec 8, 2021
abd6823
adapt ERC777 to reduce allowance before transfer
frangio Dec 8, 2021
af7425a
fix test for ERC777
frangio Dec 8, 2021
5c1aa00
Merge branch 'master' into transferfrom-allowance
frangio Dec 27, 2021
b332532
Merge branch 'master' into transferfrom-allowance
Amxx Dec 28, 2021
1a7de84
use smaller number to reduce balance
frangio Dec 29, 2021
8b3be05
simplify test description
frangio Dec 29, 2021
787cc5f
don't use deprecated expectEvents.inLogs
Amxx Dec 29, 2021
f349123
Merge remote-tracking branch 'frangio/transferfrom-allowance' into tr…
Amxx Dec 29, 2021
798992e
fix test
Amxx Dec 29, 2021
a748e2e
Merge branch 'master' into transferfrom-allowance
Amxx Dec 31, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'master' into transferfrom-allowance
  • Loading branch information
@Amxx
Amxx authored Dec 28, 2021
commit b33253218b6d09f46a179032e0ad4fca43329d7c
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
* `ERC721`: improved revert reason when transferring from wrong owner. ([#2975](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/2975))
* `Votes`: Added a base contract for vote tracking with delegation. ([#2944](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/2944))
* `ERC721Votes`: Added an extension of ERC721 enabled with vote tracking and delegation. ([#2944](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/2944))
* `ERC2771Context`: use immutable storage to store the forwarder address, no longer an issue since Solidity >=0.8.8 allows reading immutable variables in the constructor. ([#2917](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/2917))
* `ERC20`: reduce allowance before triggering transfer.

## 4.4.1 (2021-12-14)
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.