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

Fix ledger on the portal #1370

Merged
merged 11 commits into from
Aug 11, 2024
Merged

Fix ledger on the portal #1370

merged 11 commits into from
Aug 11, 2024

Conversation

bobo-k2
Copy link
Contributor

@bobo-k2 bobo-k2 commented Aug 10, 2024

Pull Request Summary

Ledger stop working on the portal. Updated Polkadot.js libraries to the latest version.

Because of some typing errors I added // @ts-nocheck to a fe places. This needs to be fixed later

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 Aug 10, 2024

Visit the preview URL for this PR (updated for commit 7412b67):

https://astar-apps--pr1370-fix-ledger-pbzkffec.web.app

(expires Sun, 18 Aug 2024 08:03:24 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: dd76fe72958fe2910fef9d53f0b4539b82b849db

@bobo-k2 bobo-k2 changed the title Fix/ledger Fix ledger on the portal Aug 11, 2024
@bobo-k2 bobo-k2 marked this pull request as ready for review August 11, 2024 06:25
@impelcrypto
Copy link
Member

impelcrypto commented Aug 11, 2024

I got this error with using normal account.

Endpoint: Astar
image

@bobo-k2
Copy link
Contributor Author

bobo-k2 commented Aug 11, 2024

I got this error with using normal account.

Fixed in 4966baf

@bobo-k2 bobo-k2 merged commit 7a1709a into release-hotfix Aug 11, 2024
6 checks passed
@bobo-k2 bobo-k2 deleted the fix/ledger branch August 11, 2024 08:53
bobo-k2 added a commit that referenced this pull request Aug 16, 2024
* hotfix: enable xcm with Bifrost (#1361)

* Fix ledger on the portal (#1370)

* disable celer bridge (#1353)

* disable cbridge

* fix statement shorter

* disable celer bridge icon too (#1354)

* Fix usePrice hooks (#1355)

* fix usePrice issue

* refactor, just using immediate is enough

* only enable immediate for h160 wallet (#1356)

* Add metadata hash and uplift to polkadot 12.0.1

* Added // @ts-nocheck to ignore some typing errors for now

* More // @ts-nocheck

* WithSignedTransaction

* Remove hash

* $api removed

* fix: removed useLedger and login toggle (#1371)

---------

Co-authored-by: Taegeon Alan Go <tae.gun7784@gmail.com>
Co-authored-by: Roy <92044428+impelcrypto@users.noreply.github.com>

* Add raw metadata to chain info (#1372)

* Add raw metadata to chain info

* Enable users to update metadata once more event if it is up to date

* Polkadot types errors fix (#1373)

* Fix polkadot types errors

* Be strict

* Polkasafe version update

---------

Co-authored-by: Roy <92044428+impelcrypto@users.noreply.github.com>
Co-authored-by: Taegeon Alan Go <tae.gun7784@gmail.com>
impelcrypto added a commit that referenced this pull request Aug 21, 2024
* hotfix: enable xcm with Bifrost (#1361)

* Fix ledger on the portal (#1370)

* disable celer bridge (#1353)

* disable cbridge

* fix statement shorter

* disable celer bridge icon too (#1354)

* Fix usePrice hooks (#1355)

* fix usePrice issue

* refactor, just using immediate is enough

* only enable immediate for h160 wallet (#1356)

* Add metadata hash and uplift to polkadot 12.0.1

* Added // @ts-nocheck to ignore some typing errors for now

* More // @ts-nocheck

* WithSignedTransaction

* Remove hash

* $api removed

* fix: removed useLedger and login toggle (#1371)

---------

Co-authored-by: Taegeon Alan Go <tae.gun7784@gmail.com>
Co-authored-by: Roy <92044428+impelcrypto@users.noreply.github.com>

* Add raw metadata to chain info (#1372)

* Add raw metadata to chain info

* Enable users to update metadata once more event if it is up to date

* Polkadot types errors fix (#1373)

* Fix polkadot types errors

* Be strict

* Polkasafe version update

* Fix for getting dApps (#1374)

---------

Co-authored-by: Roy <92044428+impelcrypto@users.noreply.github.com>
Co-authored-by: Taegeon Alan Go <tae.gun7784@gmail.com>
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