Skip to content

Remove admin and implementation getters from TransparentUpgradeableProxy #3820

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

Merged
frangio merged 15 commits into from
Nov 25, 2022
Merged

Remove admin and implementation getters from TransparentUpgradeableProxy #3820

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
5bc5eab
Remove admin and implementation getters
JulissaDantes Nov 16, 2022
07920a3
create changelog entry
JulissaDantes Nov 17, 2022
0f238ea
Include PR number
JulissaDantes Nov 17, 2022
9032c18
Update test
JulissaDantes Nov 17, 2022
7b08af9
Update test
JulissaDantes Nov 17, 2022
3974b81
Update CHANGELOG.md
JulissaDantes Nov 25, 2022
1da2602
Update ProxyAdmin tests
JulissaDantes Nov 25, 2022
6f773c7
Use helper function
JulissaDantes Nov 25, 2022
55527a0
Update documentation
JulissaDantes Nov 25, 2022
69b9662
remove extra space
frangio Nov 25, 2022
a1a636b
remove extra space
frangio Nov 25, 2022
76291fe
Add helper
JulissaDantes Nov 25, 2022
ae2e0e8
Merge branch 'remove-ifAdmin' of https://github.com/JulissaDantes/ope…
JulissaDantes Nov 25, 2022
46f866a
rename variable
frangio Nov 25, 2022
c2751e7
Merge branch 'next-v5.0' into remove-ifAdmin
frangio Nov 25, 2022
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
Include PR number
  • Loading branch information
@JulissaDantes
JulissaDantes committed Nov 17, 2022
commit 0f238ea46a63289d8411b4c161e0bcc7c61a3c02
2 changes: 1 addition & 1 deletion CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
## Unreleased (Breaking)

* `TimelockController`: Changed the role architecture to use `DEFAULT_ADMIN_ROLE` as the admin for all roles, instead of the bespoke `TIMELOCK_ADMIN_ROLE` that was used previously. This aligns with the general recommendation for `AccessControl` and makes the addition of new roles easier. Accordingly, the `admin` parameter and timelock will now be granted `DEFAULT_ADMIN_ROLE` instead of `TIMELOCK_ADMIN_ROLE`. ([#3799](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/3799))
* `TransparentUpgradeableProxy`: The `admin()` and `implementation()` getters were removed, this impacts the `ProxyAdmin` contract as well, removing the `getProxyAdmin` and `getProxyImplementation` getters. ([#](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/))
* `TransparentUpgradeableProxy`: The `admin()` and `implementation()` getters were removed, this impacts the `ProxyAdmin` contract as well, removing the `getProxyAdmin` and `getProxyImplementation` getters. ([#3820](https://github.com/OpenZeppelin/openzeppelin-contracts/pull/3820))

## Unreleased

Expand Down