Skip to content

Conversation

@vzotova
Copy link
Contributor

@vzotova vzotova commented Jun 25, 2022

No description provided.

@vzotova vzotova requested review from cygnusv and pdyraga June 25, 2022 11:47
theref
theref previously approved these changes Aug 12, 2022
Copy link
Member

@theref theref left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

Copy link
Member

@manumonti manumonti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! ✌️

manumonti
manumonti previously approved these changes Aug 17, 2022
Copy link
Member

@manumonti manumonti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! ✌️

cygnusv
cygnusv previously approved these changes Aug 17, 2022
@vzotova vzotova dismissed stale reviews from cygnusv, manumonti, and theref via 15dd79a August 21, 2022 15:43
@pdyraga
Copy link
Member

pdyraga commented Aug 24, 2022

Some context for spectators and future us: this change will allow to top-up stake from the reward merkle distributor contract in a single transaction.

@pdyraga pdyraga merged commit d11ed98 into main Aug 24, 2022
@pdyraga pdyraga deleted the permit-top-up branch August 24, 2022 09:01
@pdyraga pdyraga added this to the v1.2.0 milestone Sep 29, 2022
@cygnusv
Copy link
Member

cygnusv commented Nov 20, 2023

Some context for spectators and future us: this change will allow to top-up stake from the reward merkle distributor contract in a single transaction.

And also, once we activate rewards distribution from application contracts, it will be possible to automatically top up from there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants