Add Premint api js #271
Add Premint api js #271
Conversation
|
please also export this api in other api methods that would be useful:
|
| network: NetworkConfig; | ||
| chain: Chain; | ||
| mintFee: BigInt; | ||
| constructor(chain: Chain) { |
There was a problem hiding this comment.
thoughts on just taking a chainId instead of requiring a full chain?
There was a problem hiding this comment.
Full chain is needed in a few cases for viem to lazy-create a publicClient.
| return await response.json(); | ||
| } | ||
|
|
||
| async post(path: string, data: any) { |
There was a problem hiding this comment.
minor, but this should be extracted to a static function outside the class since it doesn't have any dependency on this, same as above with get. would clean up the api definition too
There was a problem hiding this comment.
I want to allow this to be easily overridable
There was a problem hiding this comment.
what's the reason for someone wanting to override this?
| return await response.json(); | ||
| } | ||
|
|
||
| async createPremint({ |
There was a problem hiding this comment.
if this is a premint for a new contract, the account address must match the contractAdmin.
* Add ERC7572 support for opensea. * add changesset * fix tests * fix lint
No description provided.