Skip to content
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

Allow min commitment of single state sync event #1475

Conversation

Stefan-Ethernal
Copy link
Contributor

@Stefan-Ethernal Stefan-Ethernal commented May 6, 2023

Description

Through this PR we are supporting a single state sync event per commitment. There is a real case where one deposit can be put on hold until another one happens, because, before this PR, minimum commitment size was 2.

This now helps users to quickly execute their deposit on L2, since if no other deposit happens in the given sprint, commitment with only that one state sync event will be committed to StateReceiver contract, and user can execute its deposit without the need to wait for another one.

Changes include

  • Bugfix (non-breaking change that solves an issue)
  • Hotfix (change that solves an urgent issue, and requires immediate attention)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

  • I have assigned this PR to myself
  • I have added at least 1 reviewer
  • I have added the relevant labels
  • I have updated the official documentation
  • I have added sufficient documentation in code

Testing

  • I have tested this code with the official test suite
  • I have tested this code manually

@Stefan-Ethernal Stefan-Ethernal added the bug fix Functionality that fixes a bug label May 6, 2023
@Stefan-Ethernal Stefan-Ethernal self-assigned this May 6, 2023
@Stefan-Ethernal Stefan-Ethernal force-pushed the EVM-616-fill-commitments-that-have-only-single-state-sync branch from 2c57ef0 to 25cd759 Compare May 6, 2023 06:36
@goran-ethernal goran-ethernal force-pushed the EVM-616-fill-commitments-that-have-only-single-state-sync branch from 4940574 to b6df9a2 Compare May 16, 2023 11:53
@goran-ethernal goran-ethernal self-assigned this May 16, 2023
@goran-ethernal goran-ethernal added feature New update to Polygon Edge and removed bug fix Functionality that fixes a bug labels May 16, 2023
@goran-ethernal goran-ethernal marked this pull request as ready for review May 16, 2023 12:54
@goran-ethernal goran-ethernal requested a review from a team May 16, 2023 12:54
@goran-ethernal goran-ethernal marked this pull request as draft May 16, 2023 12:55
@goran-ethernal goran-ethernal force-pushed the EVM-616-fill-commitments-that-have-only-single-state-sync branch from 877619c to 1ed3448 Compare May 23, 2023 09:41
@goran-ethernal goran-ethernal marked this pull request as ready for review May 23, 2023 09:54
@goran-ethernal goran-ethernal force-pushed the EVM-616-fill-commitments-that-have-only-single-state-sync branch from 406945c to 0a647f9 Compare May 23, 2023 09:55
Copy link
Contributor Author

@Stefan-Ethernal Stefan-Ethernal left a comment

Choose a reason for hiding this comment

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

✔️ LGTM (cannot approve it because I'm a creator of the PR).

@goran-ethernal goran-ethernal force-pushed the EVM-616-fill-commitments-that-have-only-single-state-sync branch from 0a647f9 to 1046cf4 Compare May 24, 2023 07:39
@goran-ethernal goran-ethernal merged commit 7890d28 into develop May 24, 2023
10 checks passed
@goran-ethernal goran-ethernal deleted the EVM-616-fill-commitments-that-have-only-single-state-sync branch May 24, 2023 09:34
@github-actions github-actions bot locked and limited conversation to collaborators May 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature New update to Polygon Edge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants