Skip to content
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

Re-try fetch from cbridge #1382

Merged
merged 1 commit into from
Sep 3, 2024
Merged

Re-try fetch from cbridge #1382

merged 1 commit into from
Sep 3, 2024

Conversation

bobo-k2
Copy link
Contributor

@bobo-k2 bobo-k2 commented Sep 3, 2024

Pull Request Summary

This is PR is to re-try fetching data from cbridge API. I noticed that recently first request to their API started to fail, but 2nd one works fine.

Because of that issue assets page spins forever when user is connected with EVM wallet
image

Check list

  • contains breaking changes
  • adds new feature
  • modifies existing feature (bug fix or improvements)
  • relies on other tasks
  • documentation changes
  • tested on mobile devices

Copy link

github-actions bot commented Sep 3, 2024

Visit the preview URL for this PR (updated for commit 131e369):

https://astar-apps--pr1382-fix-cbridge-fetch-c5ri5blp.web.app

(expires Tue, 10 Sep 2024 08:10:10 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: dd76fe72958fe2910fef9d53f0b4539b82b849db

@bobo-k2 bobo-k2 merged commit 61a52bc into release-hotfix Sep 3, 2024
5 of 6 checks passed
@bobo-k2 bobo-k2 deleted the fix/cbridge-fetch branch September 3, 2024 10:02
impelcrypto added a commit that referenced this pull request Sep 13, 2024
* dApps loading error fix for Shibuya (#1377)

* Tier thresholds derivation support (#1376)

* Tier thresholds derivation support

* Additional comments

* Pallet version check

* Fix: estimated realized inflation not showing (#1379)

* Re-try fetch from cbridge (#1382)

---------

Co-authored-by: Bobo <[email protected]>
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.

4 participants