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

Feat/wallet #156

Merged
merged 34 commits into from
Aug 30, 2023
Merged

Feat/wallet #156

merged 34 commits into from
Aug 30, 2023

Conversation

tomicvladan
Copy link
Collaborator

@tomicvladan tomicvladan commented Aug 17, 2023

Turning the Blossom extension into a wallet.

New features:

  • Checking balance of the current account
  • Sending ether
  • Wallet lock
  • Importing ERC20 tokens
  • Checking balance of tokens
  • Sending tokens
  • List of sent transactions
  • Providing API for dApps to interact with wallet
  • Swarm extension has become optional

Copy link
Contributor

@IgorShadurin IgorShadurin left a comment

Choose a reason for hiding this comment

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

  1. Try to enter simple eth address instead of smart contract address during token import. You can see error:
Error: call revert exception [ See: https://links.ethers.org/v5-errors-CALL_EXCEPTION ] (method="name()", data="0x", errorArgs=null, errorName=null, errorSignature=null, reason=null, code=CALL_EXCEPTION, version=abi/5.7.0)

Would be great to show something user-friendly


  1. Try to send more ETH than available, it will show also user unfriendly error.

"SELECT": "Select",
"SELECT_ADDRESS": "Choose address to send to",
"PROCEED": "Proceed",
"SENDING_TO": "Sednding to",
Copy link
Contributor

Choose a reason for hiding this comment

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

"Sending to" instead of "Sednding to"

@tomicvladan
Copy link
Collaborator Author

@IgorShadurin Changes are made in this PR. I separated wallet fixes there because the PRs are already too large.

* feat: wallet improvements

* feat: wallet improvements

* fix: wallet lock options

* fix: wallet lock mechanism

* feat: fetching account info

* feat: improve error messages ui
@IgorShadurin IgorShadurin self-requested a review August 30, 2023 09:28
@tomicvladan tomicvladan merged commit 4bb7f20 into master Aug 30, 2023
2 checks passed
@tomicvladan tomicvladan deleted the feat/wallet branch August 30, 2023 11:00
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.

None yet

3 participants