harold-nft
ee4d8380d3
Fix minor typo in IERC1155 and IERC721 docs ( #3419 )
3 years ago
GitHubPang
e633ee9ed3
Fix spelling and grammar in comments ( #3408 )
3 years ago
Mazen Khalil
3b9381dfb1
Add customizable fee receiver to ERC20FlashMint ( #3327 )
...
Co-authored-by: Mazen Khalil <mazen@immunityledger.org>
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
3 years ago
Hadrien Croubois
07b1b472c0
Improve wording consistency in code/doc ( #3365 )
3 years ago
Hadrien Croubois
a035b235b4
Release v4.6 ( #3358 )
...
* 4.6.0-rc.0
* Fix release script to only release @openzeppelin/contracts
(cherry picked from commit 2bd75a44bb
)
* make ERC2981:royaltyInfo public (#3305 )
(cherry picked from commit d2832ca7a9
)
Signed-off-by: Hadrien Croubois <hadrien.croubois@gmail.com>
* add transpilation guards to the crosschain mocks (#3306 )
(cherry picked from commit 9af5af8fff
)
Signed-off-by: Hadrien Croubois <hadrien.croubois@gmail.com>
* Fix tests on upgradeable contracts after transpilation
(cherry picked from commit 0762479dd5
)
Signed-off-by: Hadrien Croubois <hadrien.croubois@gmail.com>
* Remove unused constructor argument
(cherry picked from commit 69c3781043
)
Signed-off-by: Hadrien Croubois <hadrien.croubois@gmail.com>
* Bump minimum Solidity version for Initializable.sol to 0.8.2 (#3328 )
(cherry picked from commit cb14ea3c5c
)
* Fix update-comment script to ignore invalid tags
(cherry picked from commit 848fef5b6c
)
Signed-off-by: Hadrien Croubois <hadrien.croubois@gmail.com>
* 4.6.0
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
3 years ago
Eric Lau
c12076fb7e
Fix ERC777 link ( #3351 )
3 years ago
Francisco Giordano
dd01889434
Remove outdated documentation in ERC2981._setTokenRoyalty
3 years ago
mcIovin
731e199038
Improve docs for ERC721URIStorage._burn ( #3324 )
3 years ago
dmfxyz
d4d8d2ed97
Fix burn documentation ( #3246 )
...
Co-authored-by: xombxomb <dmfinn@wustl.edu>
3 years ago
Amin Bashiri
742e85be7c
Change zero address revert message in the balanceOf function of ERC721 and ERC1155 ( #3314 )
3 years ago
Hadrien Croubois
d2832ca7a9
make ERC2981:royaltyInfo public ( #3305 )
3 years ago
Jean Cvllr
3f49408fb6
Move event definition at the top of IERC20, IERC777 and IERC1820 ( #3228 )
3 years ago
Jeff
e7719ded56
Match IERC721 function order with EIP spec ( #3287 )
3 years ago
S E R A Y A
02fcc75bb7
Add ERC1155URIStorage ( #3210 )
...
* Add ERC721URIStorage-like extension for ERC1155
* Add tests for ERC1155URIStorage extension
* add changelog entry for ERC721URIStorage
* Fix linting errors
* Emit URI event in ERC1155URIStorage
* Remove exists check and ERC1155Supply dependency
* Fix lint error
* Overwrite ERC1155 uri method
* Update ERC1155URIStorage specs
* Fix ERC1155URIStorageMock
* Rename _setTokenURI => _setURI in ERC1155URIStorage
* Add baseURI to ERC1155URIStorage
* Move super.uri call in ERC1155URIStorage
* Clearify ERC1155URIStorage description in change log
* reorder changelog & add documentation
* improve documentation
* fix typo
Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
3 years ago
Hadrien Croubois
98716177ae
Inherit ERC20Wrapper decimals from the underlying ( #3259 )
3 years ago
Pascal Marco Caversaccio
cc1c18098c
Replace immutable with constant for _PERMIT_TYPEHASH ( #3196 )
...
* replace `immutable` with `constant` for _PERMIT_TYPEHASH
This commit is related to the following issue discussion: https://github.com/OpenZeppelin/contracts-wizard/issues/89#issuecomment-1042391318
Since Solidity version `0.6.12` the `keccak256` of string literals is treated specially and the hash is evaluated at compile time. Since the OpenZeppelin Wizard also uses `constant` for OpenZeppelin's AccessControl's roles declarations, it's good practice to make this consistent.
* Update CHANGELOG
* fix: ensure transpiler compatibility
* fix: fixing var-name-mixedcase
* prettier & lint check
Signed-off-by: Pascal Marco Caversaccio <pascal.caversaccio@hotmail.ch>
Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
3 years ago
Hadrien Croubois
f2a311dc4a
Make Votes._getVotingUnits view ( #3225 )
...
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
3 years ago
Amirhossein Banavi
62eb4568be
Optimize ERC721 _isApprovedOrOwner function ( #3248 )
3 years ago
Hadrien Croubois
f8bfa560e9
Use _spendAllowance in ERC20FlashMint ( #3226 )
...
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
3 years ago
JulissaDantes
abdb20a6bd
Change dependency source to work around Truffle limitation ( #3218 )
3 years ago
Hadrien Croubois
3e74681e77
Read allowance from overridable function in increase/decrease ( #3213 )
3 years ago
Hadrien Croubois
f6b614a7c5
Perform ERC1155.afterTokenTransfer hooks before acceptance check ( #3215 )
3 years ago
GitHubPang
458697be32
Fix typos ( #3207 )
...
Fix misspelling in ERC20 contract comments.
3 years ago
Hadrien Croubois
3dfc0a44f8
ERC1155 after token transfer hook ( #3166 )
...
* add Hooks _afterTokenTransfer
* avoid duplicate call to _asSingleton
* add changelog entry
* update changelog link to PR
* Update CHANGELOG.md
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
Co-authored-by: CryptoV8 <91189073+CryptoV8@users.noreply.github.com>
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
3 years ago
Alexander
be3c5ca504
clarify _spendAllowance natspec ( #3188 )
3 years ago
GitHubPang
4cb0926f11
Fix grammar typo in doc ( #3186 )
...
Add missing verb "are".
3 years ago
Francisco Giordano
afb20119b3
Release v4.5
3 years ago
Hadrien Croubois
c5a6cae898
Add a _spendAllowance function to ERC20 & ERC777 ( #3170 )
3 years ago
Micah Zoltu
63b466901f
Rename some ERC20 parameters to match the standard document. ( #3167 )
...
* Renames `sender` to `source`.
The naming variable was incorrect. The source of the funds is *not* necessarily (and in most cases isn't) the sender of the transaction. Also, this code has a `msgSender` which further adds confusion.
* Changes to `from/to` instead of `source`.
* Function documentation matches new names
* Changed other instances of sender/recipient to from/to.
Also changed `msgSender` to `owner` in the approval related methods.
* apply changes to IERC20.sol + minor renaming in ERC20.sol
Co-authored-by: Daniel Von Fange <daniel@leancoder.com>
Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
3 years ago
Hadrien Croubois
574f3b89e1
Add proper revert message on overflow of totalSupply during burn ( #3144 )
3 years ago
Harsh Vakharia
a81b07ce91
Fix typo in retrieval of onERC721Received selector ( #3151 )
3 years ago
Hadrien Croubois
6fb1e843cf
Make royaltyInfo(uint256 _tokenId, uint256 _salePrice) virtual ( #3133 )
...
* Make royaltyInfo(uint256 _tokenId, uint256 _salePrice) virtual
Should be cherrypicked in release 4.5
* fix lint
3 years ago
Kevin Upton
e298476a90
Simplification of ERC777's transfer & transferFrom by using _send ( #3128 )
...
* Update ERC777.sol
* Update ERC777.sol
* Update ERC777.sol
* Update ERC777.sol
* fix revert reasons
Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
3 years ago
Muhammad Anas Baig
8f70c8867e
Fix typo in ERC721.sol ( #3127 )
3 years ago
JulissaDantes
b3b83b558e
Add Slither reentrancy check in CI ( #3047 )
...
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
3 years ago
Hadrien Croubois
ecae978cb5
Make more functions virtual ( #3078 )
...
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
3 years ago
Francisco Giordano
dee772a55f
Add documentation to TokenTimelock
3 years ago
0xclaudeshannon
80d8da0564
Do not reduce approval on transferFrom if current allowance is type(uint256).max ( #3085 )
...
* add feature request #3084
* Update contracts/token/ERC20/ERC20.sol
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
* Add changelog note
* add documentation
* test unlimitted allowance and add ERC777 unlimitted allowance
* reference PR in changelog
* documentation IERC20 -> ERC20
* use asciidoc note syntax
* use asciidoc note syntax
* typo
* typos
Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
3 years ago
Martín Triay
828fe365ee
Normalize return variable names in IERC721Enumerable ( #3079 )
...
* Name missing return variable in IERC721Enumerable
For consistency among all functions.
* remove return variable names
3 years ago
JulissaDantes
a65c03bc0d
Implement Non Fungible Token Royalty (EIP2981) ( #3012 )
...
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
3 years ago
Francisco Giordano
1e815f3308
Restructure intro to ERC721 contract docs
3 years ago
Francisco Giordano
a9f994f063
Reduce ERC20 allowance before triggering transfer ( #3056 )
...
* Reduce ERC20 allowance before triggering transfer
* adapt ERC777 to reduce allowance before transfer
* fix test for ERC777
* use smaller number to reduce balance
* simplify test description
* don't use deprecated expectEvents.inLogs
* fix test
Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
3 years ago
Francisco Giordano
a0a8bbb57f
Remove note about enabling self-delegation
3 years ago
Francisco Giordano
6bd6b76d11
4.4.1
3 years ago
JulissaDantes
b42b05311b
Add ERC721Votes for NFT-based governance ( #2944 )
...
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
Co-authored-by: Hadrien Croubois <hadrien@openzeppelin.com>
3 years ago
robriks
0c858e2071
Improve revert reason in ERC721 ( #2975 )
...
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
3 years ago
Francisco Giordano
4961a51cc7
4.4.0
3 years ago
Francisco Giordano
2f2604d673
Deprecate presets in favor of Wizard ( #2986 )
3 years ago
Francisco Giordano
65ef662a2b
4.4.0-rc.1
3 years ago
Hadrien Croubois
70138680cf
Fix ERC1155 supply tracking ( #2956 )
...
Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
(cherry picked from commit 4088540aef
)
3 years ago