Skip to content

CP-9863: Fix endless spinner on Claim Rewards screen #2217

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Feb 3, 2025
Merged

Conversation

atn4z7
Copy link
Collaborator

@atn4z7 atn4z7 commented Feb 3, 2025

Description

Ticket: CP-9863

avalanchejs 4.1.2-alpha.3 adjusts the error message for insufficient nAVAX to cover fee via this commit ava-labs/avalanchejs@ef80a85. staking is not working due to us not being able to grab the missing amount. this pr adjusts the regex to match it.

Testing

please re-test all staking operations that involves transferring from C-Chain as well as claiming rewards

Checklist

Please check all that apply (if applicable)

  • I have performed a self-review of my code
  • I have verified the code works
  • I have added/updated necessary unit tests
  • I have updated the documentation

@@ -26,10 +26,12 @@ export const useCChainBaseFee = (): UseQueryResult<
retry: false,
refetchInterval: REFETCH_INTERVAL,
queryKey: ['cChainBaseFee', isDeveloperMode, cChainNetwork, avaxProvider],
enabled: !!avaxProvider && !!cChainNetwork,
queryFn: async () => {
if (!cChainNetwork || !avaxProvider) {
return Promise.reject('cChainNetwork or avaxProvider is not available')
}
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess we can remove this if condition.

@atn4z7 atn4z7 merged commit 29894be into main Feb 3, 2025
4 checks passed
@atn4z7 atn4z7 deleted the fix-staking branch February 3, 2025 17:50
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