-
Notifications
You must be signed in to change notification settings - Fork 3
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 additional chain parameters required to configure an Airnode #5
Comments
@RiVal-cz asked about if this package will export chain parameters that can help with Market payments. This issue covers that in that once it exports the block time of the chains, the Market can use a function of that as the minimum confirmations required. |
I'm happy to address this but need some more info on the fields we want to add and what the expected values should be for each existing chain |
The only way I see this issue being addressed is I ask everyone to find out all the personally-kept chain taxonomies spread across multiple environments because they won't come and tell us. Also, these properties should be auto-populated/updated because entering them manually is error-prone (especially important because we're requested to integrate a lot of chains so we want juniors to be able to this correctly) and they change over time. |
ChainAPI also needs the following values if they exist:
|
|
|
Fewer number of invocations is only safe to do with a good gas price strategy. Provider recommend will have fulfillments dropped during congestion. Apparently the Optimism fork is Mantle, so we don't expect withdrawals to work with no withdrawal remainder defined there for example. |
I see, makes sense. Base is also built on the OP stack so it might also be an issue there |
The Airnode config file has many parameters that we set differently between chains, such as ones relating to number of blocks or gas parameters. These are typically based on some chain characteristics (what is the average block time, does the provider return a block base fee, etc.) The chain objects should include both these characteristics and the (recommended) config parameters derived from these.
The text was updated successfully, but these errors were encountered: