Skip to content

Conversation

@ultraviolet10
Copy link

@ultraviolet10 ultraviolet10 commented Apr 4, 2025

TL;DR

Added support to the primodium/core package for HappyChain Sepolia network by integrating the @happy.tech/core package.

What changed?

  • Added @happy.tech/core dependency to the core package
  • Created a new happyChainSepolia chain configuration with ID 216
  • Added the chain to the chainConfigs object
  • Modified createExternalAccount to accept a custom transport parameter that uses the happyProvider. Updated AccountClientProvider to use the HappyChain provider when connecting to external accounts.

Copy link
Author

ultraviolet10 commented Apr 4, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ultraviolet10 ultraviolet10 changed the title feat: extend core package w happychain config happy primodium core Apr 4, 2025
@ultraviolet10 ultraviolet10 requested review from aodhgan and norswap April 4, 2025 08:11
@ultraviolet10 ultraviolet10 self-assigned this Apr 4, 2025
@ultraviolet10 ultraviolet10 marked this pull request as ready for review April 4, 2025 08:12
@ultraviolet10 ultraviolet10 changed the title happy primodium core happychain 🤝🏻 primodium/core Apr 4, 2025
@ultraviolet10 ultraviolet10 force-pushed the aritra/happy-primodium-core branch from b9a3c9c to a2e22e8 Compare April 10, 2025 10:19
},
};

export const happyChainSepolia: ChainConfig = {
Copy link

Choose a reason for hiding this comment

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

i think we might export this from @happy.tech/core ?

try {
const txHash = await txPromise;
await waitForTransaction(txHash);
// await waitForTransaction(txHash); // doesn't work
Copy link

Choose a reason for hiding this comment

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

why?

Base automatically changed from aritra/add-happychain-config to main April 11, 2025 06:32
@ultraviolet10 ultraviolet10 force-pushed the aritra/happy-primodium-core branch from 9c094d5 to 32364c4 Compare April 11, 2025 07:10
@ultraviolet10 ultraviolet10 merged commit bf158fc into main Apr 11, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants