Skip to content

Conversation

@djhahe
Copy link
Contributor

@djhahe djhahe commented Nov 25, 2023

Summary (Please recap what you changed or fixed)

Checklist (If you won't pass this checklist please comment why)

  • I removed all commented or unnecessary code.
  • Provide the screenshots due to UI changed or bug fixed
  • My code has covered these test cases (please list down your tested cases):

djhahe and others added 6 commits October 27, 2023 12:55
* Feat: #367 Select and switch dapp account (#369)

* Feat: Allow user transfer NFT (#350)

* feat: allow user transfer nft

* feat: show button if isTransferable true

* chore: wrong name

* chore: fix wrong condition

* refactor: clean code for nft transfer

* test: update wrong ut (#366)

* feat: #367 allow user select account and switch dapp account

* refactor: layouts and clean code

* refactor: change main action to right

* refactor: remove unselected account when user connecting

* fix: can not set selected wallet

* fix: get config with selector (#364)

* fix: #373 should validate network fee (#374)

* Fix: #317 Get config data with network mode (#354)

* fix: #317 get config data with network mode

* refactor: #317 get config with use configuration

* test: update ut for SendReceive component with mocking

* chore: revert package command

* refactor: useConfiguration hook

* chore: remove eslintcache

* update version

* Fix: User can connect when account deleted (#376)

* update version

* fix: user can connect when account deleted

---------

Co-authored-by: kiennguyen <>

* chore: lint import

---------

Co-authored-by: Liam <20489407+hoailinh1210@users.noreply.github.com>
Co-authored-by: kiennguyen <>
Co-authored-by: hoailinh1210 <hoailinh1210@gmail.com>
Co-authored-by: kiennguyen <>
* show assets detail chart

* fix issue on token details page

* fix unit test

* fix code smell

---------

Co-authored-by: kiennguyen <>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

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.

3 participants