Skip to content

Feat/add 0g support p3 1 - #9760

Draft
maxime-oe wants to merge 5 commits into
mainfrom
feat/add-0g-support-p3_1
Draft

Feat/add 0g support p3 1#9760
maxime-oe wants to merge 5 commits into
mainfrom
feat/add-0g-support-p3_1

Conversation

@maxime-oe

Copy link
Copy Markdown
Contributor

Explanation

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Adds 0G to the two Multicall3 contract-address maps
(assets-controllers/src/multicall.ts and assets-controller's
MulticallClient.ts), using the canonical mds1/multicall3 deployment
(0xcA11bde05977b3631167028862bE2a173976CA11) -- live-verified this
session against the primary RPC and both fallbacks, byte-identical
bytecode across all three, unlike Somnia which needed a non-canonical
chain-specific deployment.

Adds 0G to SPOT_PRICES_SUPPORT_INFO in codefi-v2.ts with its dedicated
slip44:1111116661 identifier (no real SLIP-44 registry entry, not
ETH-native, so not slip44:60) -- matching chain-input.yaml's Phase 0
native-asset convention.

Changelog PR links use a PR_NUMBER placeholder -- fill in with the
real PR number once opened.

Base network enablement (0G p3_1 -- core's price + multicall wiring,
one Base-enablement PR per the skill's core exception).
PR #9760 is now open; replace the PR_NUMBER placeholder with the real
number/link, matching the Arc/Somnia precedent (commits b204ff4,
51d85df).
assets-controller (singular) changes are not required and are not
expected to ever be required for a plain Base enablement's Multicall
wiring -- only assets-controllers (plural)'s multicall.ts matters.
Base network enablement skill updated accordingly.
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.

1 participant