Skip to content
This repository was archived by the owner on Mar 5, 2025. It is now read-only.

Conversation

@luu-alex
Copy link
Contributor

@luu-alex luu-alex commented Jun 24, 2022

#4969

Description

This PR is for the documentation of both wallets and account.
It also includes some bug fixes I discovered when making the documentation:

  • fixing the return type so that wallet.add returns the wallet instead of a boolean
  • Adding account.wallet to the Web3EthInterface so wallet can be used
  • Fixing a bug that the web3 package would produce an src file instead of just the compiled files

@jdevcs jdevcs added the 4.x 4.0 related label Jun 28, 2022
@jdevcs
Copy link
Contributor

jdevcs commented Jun 29, 2022

LGTM, except some minor changes.

@luu-alex luu-alex requested a review from jdevcs June 29, 2022 20:24
@luu-alex
Copy link
Contributor Author

created issue for function that is not returning expected type.

@luu-alex luu-alex merged commit 25dd6f8 into 4.x Jun 30, 2022
@luu-alex luu-alex deleted the 4969/account-wallet-docs branch June 30, 2022 16:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

4.x 4.0 related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants