Skip to content

Add TipCollectionToggler contract - #439

Open
TucksonDev wants to merge 7 commits into
developfrom
add-tip-collection-toggler
Open

Add TipCollectionToggler contract#439
TucksonDev wants to merge 7 commits into
developfrom
add-tip-collection-toggler

Conversation

@TucksonDev

@TucksonDev TucksonDev commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

This PR adds a TipCollectionToggler contract that allows a trusted party to enable/disable tip collection in a chain (available from ArbOS 60), during a specified time. This contract should be added as a chain owner.

It also updates the pin of the precompiles submodule to v60.0.0, so it includes the ArbOwner method that needs to be called.

@TucksonDev
TucksonDev changed the base branch from main to develop June 30, 2026 11:36
@TucksonDev
TucksonDev requested a review from godzillaba June 30, 2026 11:55
Comment thread src/chain/TipCollectionToggler.sol Fixed

@godzillaba godzillaba left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! just a nit about events

we should probably wait for a precompiles release before merging to develop though

Comment thread src/chain/TipCollectionToggler.sol
@TucksonDev
TucksonDev requested a review from godzillaba July 15, 2026 12:04

@godzillaba godzillaba left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM assuming CI passes

I pushed one commit to hide a slither issue

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.

3 participants