-
Notifications
You must be signed in to change notification settings - Fork 12.4k
Add Ownable2Step extension with 2-step transfer #3620
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
fccb61f
Change _owner to internal
heldersepu 918ab11
Revert change to _owner variable
heldersepu 00aae11
New contract SafeOwnable
heldersepu 16f6014
SafeOwnableMock
heldersepu fbc33a8
testing the new SafeOwnable contract
heldersepu 229b1d0
format code
heldersepu 4b3369a
Add entry to changelog
heldersepu 6c7325d
Fix error 'safe_ownable' is not in camel case
heldersepu 80b5260
Fix mix of tabs and spaces
heldersepu db4af81
Allow address(0) as new owner
heldersepu c21d53d
Merge branch 'master' into master
heldersepu 2530cd5
Rename _newOwner to _pendingOwner
heldersepu de9beed
Remove unused const
heldersepu c06b773
Remove unused constants
heldersepu 3279163
override _transferOwnership to delete _pendingOwner
heldersepu d995d7c
Update contracts/access/SafeOwnable.sol
heldersepu b12b1e7
Update contracts/access/SafeOwnable.sol
heldersepu 8d62986
format code
heldersepu 320aa21
Add test to forceTransferOwnership
heldersepu fbf41df
Add check on pendingOwner
heldersepu 20aa36c
Merge branch 'master' into master
heldersepu 2ba7878
Merge branch 'master' into master
heldersepu 9bbeb47
Rename to Ownable2Step
heldersepu 16d5c16
Rename mock to Ownable2StepMock
heldersepu baf063e
Update CHANGELOG.md
heldersepu adcf9aa
Update contracts/access/Ownable2Step.sol
heldersepu 0b46f49
check the event arguments
heldersepu 50ca6bc
Merge branch 'master' of https://github.com/heldersepu/openzeppelin-c…
heldersepu f22673d
format code
heldersepu ac7e601
Add check pending owner resets on force transfer
heldersepu b781427
Update Ownable2Step.test.js
heldersepu 81c1f6c
Update contracts/access/Ownable2Step.sol
heldersepu a36cc8e
Update contracts/access/Ownable2Step.sol
heldersepu b0e363a
Update contracts/access/Ownable2Step.sol
heldersepu 8ff0621
Add previousOwner to event OwnershipTransferStarted
heldersepu ff44765
Merge branch 'master' into master
heldersepu 84f3b9f
Remove forceTransferOwnership
heldersepu fca0901
Merge branch 'master' into master
frangio File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Add check pending owner resets on force transfer
- Loading branch information
commit ac7e601d3194dc41683625e6ac5b104dacd47836
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.