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

disable celer bridge #1353

Merged
merged 2 commits into from
Jul 11, 2024
Merged

disable celer bridge #1353

merged 2 commits into from
Jul 11, 2024

Conversation

gtg7784
Copy link
Member

@gtg7784 gtg7784 commented Jul 11, 2024

Pull Request Summary

disable celer bridge, https://x.com/CelerNetwork/status/1811368611828953312

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
image

This pull request makes the following changes:

Fixes

  • Disable cbridge

@gtg7784 gtg7784 changed the base branch from main to release-hotfix July 11, 2024 13:07
Copy link

github-actions bot commented Jul 11, 2024

Visit the preview URL for this PR (updated for commit 8b0790b):

https://astar-apps--pr1353-disable-cbridge-ifq5wzey.web.app

(expires Thu, 18 Jul 2024 14:05:15 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: dd76fe72958fe2910fef9d53f0b4539b82b849db

@gtg7784 gtg7784 changed the title disable cbridge disable celer bridge Jul 11, 2024
@@ -241,6 +244,10 @@ export default defineComponent({
return isH160.value && (isAstar.value || isAstarZkEvm.value);
});

const isEnableCelerBridge = computed<boolean>(() => {
Copy link
Contributor

Choose a reason for hiding this comment

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

You can make this statemen shoreter

const isEnableCelerBridge = computed<boolean>(() => false);

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, thank you! fixed in 8b0790b commit

@gtg7784 gtg7784 merged commit e86279c into release-hotfix Jul 11, 2024
4 checks passed
@gtg7784 gtg7784 deleted the disable-cbridge branch July 11, 2024 13:57
bobo-k2 added a commit that referenced this pull request Jul 19, 2024
* 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)

* Fetch account identity fix (#1359)

---------

Co-authored-by: Taegeon Alan Go <[email protected]>
bobo-k2 added a commit that referenced this pull request Aug 11, 2024
* 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 <[email protected]>
Co-authored-by: Roy <[email protected]>
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 <[email protected]>
Co-authored-by: Roy <[email protected]>

* 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 <[email protected]>
Co-authored-by: Taegeon Alan Go <[email protected]>
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 <[email protected]>
Co-authored-by: Roy <[email protected]>

* 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 <[email protected]>
Co-authored-by: Taegeon Alan Go <[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.

2 participants