-
Notifications
You must be signed in to change notification settings - Fork 72
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
AN-374: Creating an example that will fail #940
Conversation
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.
You are missing the Requester contract in this PR, did you forget to commit it?
I am not sure about the changes in yarn.lock, what is your yarn version?
Hi @Siegrift, I don't really know why the requester contract is missing; I'll investigate and recommit. |
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.
Also, remove package-lock.json
as it is not needed.
Co-authored-by: Emanuel Tesař <[email protected]>
return encode([{ name: 'coinId', type: 'string32', value: coinId }]); | ||
}; | ||
|
||
export const printResponse = async (requestId: string) => { |
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.
This function is not needed (will never be called since the request fails) but I am OK with it staying
This pull request creates a new Airnode example that is intended to fail. I only made a minor alteration to the coingecko example in all the necessary places and documented the behaviour in the readme.