-
Notifications
You must be signed in to change notification settings - Fork 9
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
ABW-1941: Make ansharnet use RCnet V2 dashboard #640
ABW-1941: Make ansharnet use RCnet V2 dashboard #640
Conversation
@@ -33,7 +33,7 @@ extension LedgerIdentifiable { | |||
public var address: String { | |||
switch self { | |||
case let .transaction(txId): | |||
return txId.asStr() | |||
return txId.hex |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@maciek-rdx why do we need this change? is it that the dashboard does not support txId.asStr()
lookups?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmmm, I'm not aware of any dashboard-specific considerations... I just adopted what we agreed upon here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I'm not mistaken asStr will provide the HRP version of the tx id
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems that the dashboard cannot handle the tx id properly...
Jira ticket: here
Description
Check Jira ticket
How to test
Check Jira ticket
Screenshot
PR submission checklist