Skip to content

Staking requestAuthorizationDecrease function contract update #99

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
merged 4 commits into from
Aug 24, 2022
Merged

Staking requestAuthorizationDecrease function contract update #99

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
26dce2a
Staking requestAuthorizationDecrease function contract update
pdyraga Apr 28, 2022
ac6983a
Merge branch 'main' into ackchyually
pdyraga Aug 19, 2022
bc372c7
Merge branch 'main' into ackchyually
cygnusv Aug 23, 2022
631989e
Merge branch 'main' into ackchyually
cygnusv Aug 24, 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
3 changes: 3 additions & 0 deletions contracts/staking/IApplication.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,9 @@ interface IApplication {
/// pending decrease and respond to the staking contract with
/// `approveAuthorizationDecrease` at its discretion. It may
/// happen right away but it also may happen several months later.
/// If there is already a pending authorization decrease request
/// for the application, and the application does not agree for
/// overwriting it, the function should revert.
function authorizationDecreaseRequested(
address stakingProvider,
uint96 fromAmount,
Expand Down
4 changes: 3 additions & 1 deletion contracts/staking/IStaking.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,9 @@ interface IStaking {
/// it happens depends on the application. Can only be called by the
/// given staking provider’s authorizer. Overwrites pending
/// authorization decrease for the given staking provider and
/// application.
/// application if the application agrees for that. If the
/// application does not agree for overwriting, the function
/// reverts.
/// @dev Calls `authorizationDecreaseRequested(address stakingProvider, uint256 amount)`
/// on the given application. See `IApplication`.
function requestAuthorizationDecrease(
Expand Down
4 changes: 3 additions & 1 deletion contracts/staking/TokenStaking.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1281,7 +1281,9 @@ contract TokenStaking is Initializable, IStaking, Checkpoints {
/// it happens depends on the application. Can only be called by the
/// given staking provider’s authorizer. Overwrites pending
/// authorization decrease for the given staking provider and
/// application.
/// application if the application agrees for that. If the
/// application does not agree for overwriting, the function
/// reverts.
/// @dev Calls `authorizationDecreaseRequested` callback on the given
/// application. See `IApplication`.
function requestAuthorizationDecrease(
Expand Down