-
Notifications
You must be signed in to change notification settings - Fork 51
Conversation
Your Render PR Server URL is https://d3n-scan-pr-1963.onrender.com. Follow its progress at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
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.
Should we add the test to GitHub action?
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Hmm.. I'm not entirely sure here. I guess we should, but in that case, we should also use mock server requests instead of the real requests to GuanYu devnet we're doing right now. What do you think? @sorawit |
Yes. I think if we are to add it to github action, better be tests that don't rely on external API calls. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
add new line
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
return this.getRequestID(broadcastResponse.txhash) | ||
} | ||
|
||
async getRequestID(txHash) { |
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.
I saw retryTimeout = 200 at "getRequestProof". So should we add it here as well ?
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.
Nope
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Your Render PR Server at https://d3n-scan-pr-1963.onrender.com is now live! View it on your dashboard at https://dashboard.render.com/static/srv-bri88mh8vr20oh4vgj60. |
Note: the old
bandchain.js
branch got messed up due to an incorrect rebase. This PR is for the new branch with the same content.