-
Notifications
You must be signed in to change notification settings - Fork 1
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
fix add on-contract-event
#304
Conversation
import { createMeridianContract, provider } from './ie-contract.js' | ||
import { onContractEvent } from 'on-contract-event' | ||
import { RPC_URL, GLIF_TOKEN } from '../spark-publish/ie-contract-config.js' |
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'd like to keep only one place where we are importing from spark-publish.
Would you mind re-exporting these constants in ./ie-contract.js
?
import { createMeridianContract, provider } from './ie-contract.js' | |
import { onContractEvent } from 'on-contract-event' | |
import { RPC_URL, GLIF_TOKEN } from '../spark-publish/ie-contract-config.js' | |
import { RPC_URL, GLIF_TOKEN, createMeridianContract, provider } from './ie-contract.js' | |
import { onContractEvent } from 'on-contract-event' |
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.
Feel free to change the imports in a follow-up pull request.
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 think we can fix the problem by adding polling: true to the JsonRpcApiProvider options.
https://docs.ethers.org/v6/api/providers/jsonrpc/#JsonRpcApiProviderOptions
We don't need this change anymore, see #306 (comment) |
Blocked by filecoin-station/on-contract-event#3