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 alchemy provider constructor doc with updated config #293

Merged
merged 1 commit into from
Dec 1, 2023

Conversation

denniswon
Copy link
Contributor

@denniswon denniswon commented Dec 1, 2023

Pull Request Checklist

  • Did you add new tests and confirm existing tests pass? (yarn test)
  • Did you update relevant docs? (docs are found in the site folder, and guidelines for updating/adding docs can be found in the contribution guide)
  • Do your commits follow the Conventional Commits standard?
  • Does your PR title also follow the Conventional Commits standard?
  • If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
  • Did you run lint (yarn lint:check) and fix any issues? (yarn lint:fix)
  • Is the base branch you're merging into development and not main?
  • Did you follow the contribution guidelines?

PR-Codex overview

Detailed summary

  • Updated the opts parameter type in the constructor method of the SmartAccountProvider class in aa-alchemy/provider/constructor.md to SmartAccountProviderOpts.
  • Added a new feeOptions parameter in the constructor method of the SmartAccountProvider class in aa-alchemy/provider/constructor.md to set user operation fee options.
  • Added new fields maxFeePerGas, maxPriorityFeePerGas, callGasLimit, verificationGasLimit, and preVerificationGas in the opts parameter of the constructor method of the SmartAccountProvider class in aa-alchemy/provider/constructor.md.
  • Added a new note explaining the usage of custom middleware and handling fee options in aa-core/provider/constructor.md.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@avasisht23 avasisht23 merged commit 77ada09 into development Dec 1, 2023
6 checks passed
@avasisht23 avasisht23 deleted the denniswon/alchemy-provider-doc branch December 1, 2023 17:32
@moldy530 moldy530 mentioned this pull request Dec 1, 2023
8 tasks
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