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

evm: only allow owner to unpause #444

Merged
merged 2 commits into from
May 13, 2024
Merged

evm: only allow owner to unpause #444

merged 2 commits into from
May 13, 2024

Conversation

nik-suri
Copy link
Collaborator

@nik-suri nik-suri commented May 8, 2024

No description provided.

@nik-suri nik-suri requested review from RahulMaganti47 and djb15 May 8, 2024 22:51
@nik-suri nik-suri added evm Change to EVM Contracts P0 enhancement New feature or request labels May 8, 2024
Copy link
Contributor

@RahulMaganti47 RahulMaganti47 left a comment

Choose a reason for hiding this comment

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

lgtm

@nik-suri nik-suri merged commit c8aab39 into main May 13, 2024
5 checks passed
@nik-suri nik-suri deleted the owner-unpause branch May 13, 2024 20:58
bruce-riley pushed a commit to wormholelabs-xyz/native-token-transfers that referenced this pull request Sep 12, 2024
* evm: only allow owner to unpause

* Extend test to assert owner can still unpause
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request evm Change to EVM Contracts P0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants