Skip to content

Conversation

@andygruening
Copy link
Contributor

@andygruening andygruening commented Aug 26, 2025

It continues to get harder to add new chains, when the current dictionaries grow. Instead of defining each chain in 7 different dictionaries, let's create an array of ChainConfig classes. When the static class is created, the existing dictionaries are automatically populated.

Going from 400 lines of code to 100
Existing integrations won't break.

This PR includes support for Incentiv Testnet v2

Version Increment

Please ensure you have incremented the package version in the package.json as necessary.

  • I have incremented the package.json according to semantic versioning
  • No version increment is needed; the change does not impact SDK or Sample code/assets

Docs Checklist

Please ensure you have addressed documentation updates if needed as part of this PR:

  • I have created a separate PR on the sequence docs repository for documentation updates: Link to docs PR
  • No documentation update is needed for this change.

@andygruening andygruening requested a review from a team August 26, 2025 19:27
@andygruening andygruening requested a review from a team as a code owner August 26, 2025 19:27
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Aug 26, 2025

Deploying sequence-unity with  Cloudflare Pages  Cloudflare Pages

Latest commit: b0abe09
Status: ✅  Deploy successful!
Preview URL: https://616c9fad.sequence-unity.pages.dev
Branch Preview URL: https://feature-new-chain-config.sequence-unity.pages.dev

View logs

@andygruening andygruening changed the title created new class-based chain config created new class-based chain config, incentiv testnet v2 Sep 4, 2025
@andygruening andygruening merged commit 19da966 into master Sep 4, 2025
4 checks passed
@andygruening andygruening deleted the Feature/new-chain-config branch September 4, 2025 15:22
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