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(statics): update ADA block explorer url #3690

Merged

Conversation

yashrohatgi-bitgo
Copy link
Contributor

@yashrohatgi-bitgo yashrohatgi-bitgo commented Jun 23, 2023

  • explorer url was not linking to transactions correctly, added to url accordingly

Ticket: CE-985

@yashrohatgi-bitgo yashrohatgi-bitgo marked this pull request as ready for review June 23, 2023 14:31
@yashrohatgi-bitgo yashrohatgi-bitgo requested review from a team as code owners June 23, 2023 14:31
CapnMigraine
CapnMigraine previously approved these changes Jun 23, 2023
@@ -120,7 +120,7 @@ class Ada extends Mainnet implements AdaNetwork {
utxolibName = 'cardano';
poolDeposit = 500000000;
stakeKeyDeposit = 2000000;
explorerUrl = 'https://explorer.cardano.org/en/';
explorerUrl = 'https://explorer.cardano.org/en/transaction?id=';
Copy link
Contributor

@alebusse alebusse Jun 23, 2023

Choose a reason for hiding this comment

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

This gonna is gonna be parsed by the UI as
https://explorer.cardano.org/en/transaction + txId + ?id=

https://github.com/BitGo/bitgo-microservices/blob/b51dc7e69d393f293ee3dbb14a7530488555a226/packages/client-app-dashboard/app/shared/api/utils.ts#L472

Copy link
Contributor

Choose a reason for hiding this comment

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

This should be fixed in that repo, not here as this is the base definition.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah I see that now. Not sure how I'd work around this though aside from hard coding something in the UI

@yashrohatgi-bitgo yashrohatgi-bitgo merged commit f748b26 into master Jun 27, 2023
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.

4 participants