Skip to content
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

Docs update #231

Merged
merged 6 commits into from
Apr 11, 2022
Merged

Docs update #231

merged 6 commits into from
Apr 11, 2022

Conversation

Vectorized
Copy link
Collaborator

@Vectorized Vectorized commented Apr 8, 2022

  • Upgradeable docs
  • ERC721AQueries docs
  • ERC721APausable docs
  • Various small edits

To be merged just before publishing next version.

multiple smaller scans if the collection is large enough to
cause an out-of-gas error.

This function should work fine for 10k PFP collections.
Copy link
Contributor

@Austinhs Austinhs Apr 8, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thoughts on maybe adding a similar comment for a size that would not work?

Copy link
Contributor

@Austinhs Austinhs Apr 8, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something like:

Should be avoided for collections > 30K (arbitrary number, btw), and should NOT be used for dynamic supplies like breeding.

(If we don't wanna go through the testing to figure out the limitation, might just mention that its only been tested on 10K collections)

Copy link
Collaborator Author

@Vectorized Vectorized Apr 8, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am purposely vague about the limit, because different RPC providers may have different gas limits on view functions.

Also, these limits may change over time (e.g. Ethereum's max block gas limit).

@Vectorized Vectorized merged commit f1f202c into chiru-labs:main Apr 11, 2022
@Vectorized Vectorized deleted the docsUpdate branch April 12, 2022 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants