Skip to content

Adds checks for squidrouter swap transaction value.#585

Merged
gianfra-t merged 4 commits into
stagingfrom
increase-moonbeam-funding
Apr 23, 2025
Merged

Adds checks for squidrouter swap transaction value.#585
gianfra-t merged 4 commits into
stagingfrom
increase-moonbeam-funding

Conversation

@gianfra-t
Copy link
Copy Markdown
Contributor

@gianfra-t gianfra-t commented Apr 23, 2025

Changes

  • Increase ephemeral funding.
  • Will not create quote if ephemeral funding constant is not be enough to pay for the squidrouter transaction.

@gianfra-t gianfra-t requested a review from ebma April 23, 2025 12:26
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 23, 2025

Deploy Preview for pendulum-pay ready!

Name Link
🔨 Latest commit
🔍 Latest deploy log https://app.netlify.com/sites/pendulum-pay/deploys/68090194b6254f0f41fa69cf
😎 Deploy Preview https://deploy-preview-585--pendulum-pay.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Comment thread api/src/constants/constants.ts Outdated
Comment on lines +78 to +83
export function getMoonbeamFundingData(decimals: number, boosted: boolean = false): {
fundingAmountRaw: string;
walletClient: ReturnType<typeof createMoonbeamClientsAndConfig>['walletClient'];
publicClient: ReturnType<typeof createMoonbeamClientsAndConfig>['publicClient'];
} {
const fundingAmountUnits = Big(MOONBEAM_EPHEMERAL_STARTING_BALANCE_UNITS);
const fundingAmountUnits = boosted ? Big(MOONBEAM_EPHEMERAL_STARTING_BALANCE_UNITS_ETHEREUM) : Big(MOONBEAM_EPHEMERAL_STARTING_BALANCE_UNITS);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

boosted reminds me of squidrouter. But that's not entirely true so let's change the name. Maybe call it largeFunding: boolean = false?

@gianfra-t gianfra-t merged commit 097b8ba into staging Apr 23, 2025
5 checks passed
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.

2 participants