-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add Sapphire on testnet #279
Conversation
01ea0e9
to
c40040a
Compare
fa80afa
to
c57582d
Compare
c57582d
to
2d22887
Compare
@@ -808,7 +808,7 @@ class APIService { | |||
let u8Hash = await oasis.hash.hash(oasis.misc.toCBOR(txWrapper.unverifiedTransaction)) | |||
let hash = oasis.misc.toHex(u8Hash) | |||
let config = getRuntimeConfig(params.runtimeId) | |||
if (hash && config.accountType === RUNTIME_ACCOUNT_TYPE.EVM) { |
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.
What was wrong with this approach?
Also does that PR need to address another use cases like
let isEmerald = runtimeConfig.accountType === RUNTIME_ACCOUNT_TYPE.EVM
where now we have multiple EVMc paratimes?
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.
Do you mean with if (hash && config.accountType === RUNTIME_ACCOUNT_TYPE.EVM && config.runtimeName === "Emerald") {
approach? There's no difference
I think all other conditions are okay. (but isEmerald
is a bad name - and there's already isEvm
too :/ )
No description provided.