-
Notifications
You must be signed in to change notification settings - Fork 586
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
update NewPacket API to use new Timeout type #3600
Comments
@charleenfei can you please explain about new TImeout type? is it UpgradeTimeout or anything else? |
Is this issue still available? |
Hi @vishal-kanna, @hieuvubk -- sorry for the delay in reply. The new Please note that if either of you would like to work on this this issue, the PR should be branched off of |
@charleenfei can I work on this issue? |
See here for reasoning to close the issue. |
Summary of Bug
Currently, the
NewPacket
API expects separate timeout height and timeout timestamp parameters. We should update this to use the newTimeout
type.For Admin Use
The text was updated successfully, but these errors were encountered: