Skip to content

fix(ctb): bug in L2 genesis script#3068

Merged
mergify[bot] merged 2 commits intodevelopfrom
sc/ctb-fix-genesis-l2
Jul 19, 2022
Merged

fix(ctb): bug in L2 genesis script#3068
mergify[bot] merged 2 commits intodevelopfrom
sc/ctb-fix-genesis-l2

Conversation

@smartcontracts
Copy link
Contributor

Description
Fixes a bug in the Bedrock L2 genesis script where storage slots were
being set in the implementation rather than the proxy, which (obviously)
broke various things that the SDK relies on.

@changeset-bot
Copy link

changeset-bot bot commented Jul 19, 2022

🦋 Changeset detected

Latest commit: 2bfbf3f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/contracts-bedrock Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Fixes a bug in the Bedrock L2 genesis script where storage slots were
being set in the implementation rather than the proxy, which (obviously)
broke various things that the SDK relies on.
@smartcontracts smartcontracts force-pushed the sc/ctb-fix-genesis-l2 branch from 6e0d167 to f606790 Compare July 19, 2022 13:44
@github-actions github-actions bot added 2-reviewers A-pkg-contracts-bedrock Area: packages/contracts-bedrock labels Jul 19, 2022
@mergify
Copy link
Contributor

mergify bot commented Jul 19, 2022

This PR has been added to the merge queue, and will be merged soon.

@mergify
Copy link
Contributor

mergify bot commented Jul 19, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify mergify bot merged commit 5e11313 into develop Jul 19, 2022
@mergify mergify bot deleted the sc/ctb-fix-genesis-l2 branch July 19, 2022 16:40
@mergify mergify bot removed the on-merge-train label Jul 19, 2022
sam-goldman pushed a commit that referenced this pull request Sep 15, 2022
Fixes a bug in the Bedrock L2 genesis script where storage slots were
being set in the implementation rather than the proxy, which (obviously)
broke various things that the SDK relies on.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-pkg-contracts-bedrock Area: packages/contracts-bedrock

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants