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

Fix for dApps selection during staking #1387

Merged
merged 2 commits into from
Sep 24, 2024
Merged

Conversation

bobo-k2
Copy link
Contributor

@bobo-k2 bobo-k2 commented Sep 19, 2024

Pull Request Summary

There were bugs when searching or selecting dApps to vote on. I also applied some fixes suggested by Biome (prefix type import and enum init)

  1. if I search for 1 dapps using the SEARCH button. I find it, I select it. Then I want to select another dApp so I delete the name of the previous dapp to write the new search. At that point, the first dApp I had selected changes. (example: if I had selected VLS as the first dapp, after canceling the search I find myself selected as the first dApps destore.)
image image
  1. if I search a Dapp by [CHOOSE dAPP] section, after that I click to [DEFI]. All defi dapps is showed. Now I select 1 or 2 dApps from that list. Now If I want to add one more dApps using the [SEARCH] option, I write name, I find the dapps but the system loose the correct selections of the previous dapps.

Check list

  • contains breaking changes
  • adds new feature
  • modifies existing feature (bug fix or improvements)
  • relies on other tasks
  • documentation changes
  • tested on mobile devices

Copy link

Visit the preview URL for this PR (updated for commit a16a42b):

https://astar-apps--pr1387-fix-selected-dapp-in-c7540tj9.web.app

(expires Thu, 26 Sep 2024 13:05:06 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: dd76fe72958fe2910fef9d53f0b4539b82b849db

Copy link
Member

@gtg7784 gtg7784 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@impelcrypto impelcrypto left a comment

Choose a reason for hiding this comment

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

LGTM

@bobo-k2 bobo-k2 merged commit 452cc02 into release-hotfix Sep 24, 2024
6 checks passed
@bobo-k2 bobo-k2 deleted the fix/selected-dapp-index branch September 24, 2024 07:28
@bobo-k2 bobo-k2 changed the title Fix/selected dapp index Fix for dApps selection during staking Sep 24, 2024
impelcrypto pushed a commit that referenced this pull request Sep 30, 2024
* Remove Age of Chronos promotion (#1386)

* Fetch block time from chain (#1384)

* Fetch block time to store

* Apr calculation fix

* Type fix

* PR comments fix and code cleanup

* Fix/selected dapp index (#1387)

* Voting wizard search bug fix

* Keep selected dApps while switching categories

* APR calculation fix to use block time for a given block (#1389)

* APR calculation fix to use block time for a given block

* Small fix
impelcrypto pushed a commit that referenced this pull request Oct 1, 2024
* Remove Age of Chronos promotion (#1386)

* Fetch block time from chain (#1384)

* Fetch block time to store

* Apr calculation fix

* Type fix

* PR comments fix and code cleanup

* Fix/selected dapp index (#1387)

* Voting wizard search bug fix

* Keep selected dApps while switching categories

* APR calculation fix to use block time for a given block (#1389)

* APR calculation fix to use block time for a given block

* Small fix

* Fix for re-staking on unregistered dApps (#1392)
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