-
Notifications
You must be signed in to change notification settings - Fork 29
Conversation
00ea90f
to
65e1ea9
Compare
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.
Nice 👍 only very minor rephrasing suggestions from me.
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.
In 0.9. we also have Sepolia network available for Airnodes to use. I don't think this needs to be mentioned in the migration guide though.
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 recall @dcroote mentioned some issues with the 0.7.0 to 0.8.0
guide by Andre. I will take a look into them after merging which is when I will add your guides to the sidebars.
- #4 - might be worth mentioning rrp[n].cacheResponses is a new, required field
- Refer to airnode-admin wherever applicable #5 (both in the summary and the Details): 0.8.0 uses oisFormat "1.1.1" rather than "1.1.2" (see here for example). There is a bit of debate on how to handle the coupling of Airnode and OIS going forward though (for which we might need a 4th opinion 😅).
- Minor typo "0.8.9" at the end
- corsOrigins was added for HTTP gateways (relevant config.json changes here) in #1357.
- relayRequestId was added as a supported security scheme. Functionality implemented with an example of use in #1286 (though between that PR and now, OIS was moved into its own repo).
@wkande - Andre incorporated all of my changes before @Siegrift copied it for this PR so all set. @Siegrift - I think we should mention Sepolia is available and that we're deprecating some testnets (api3dao/airnode#1476), including why we're doing it. This way we can point users (e.g. from Discord) to one piece of information about the change. |
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.
Requesting updates per my above comment
Ok. Added in last commit. Please either @wkande or @dcroote take a look at wording and make any necessary updates. Feel free to merge it afterwards. |
Visit the preview URL for this PR (updated for commit eb302d3): https://api3-docs--pr1056-migration-guides-gnvlc8mt.web.app (expires Thu, 06 Oct 2022 05:01:29 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 |
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.
Closes api3dao/airnode#1448