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: Add validators tab to account address page #1183

Merged
merged 7 commits into from
Feb 26, 2024

Conversation

brancoder
Copy link
Collaborator

Description of change

fixes #1082

Type of change

  • Enhancement (a non-breaking change which adds functionality)

Change checklist

  • My code follows the contribution guidelines for this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@brancoder brancoder linked an issue Feb 26, 2024 that may be closed by this pull request
@@ -88,6 +93,12 @@ const buildAccountAddressTabsOptions = (
isLoading: isCongestionLoading,
infoContent: bicMessage,
},
[ACCOUNT_TABS.Validation]: {
disabled: !hasValidatorDetails,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe we should have stricter criteria for "disabled/hidden" for this one, as it seams like all addresses "have validation details". One the ones i checked it looks like this:
image

Are cases like this something that we want to show ? @begonaalvarezd

Copy link
Member

Choose a reason for hiding this comment

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

I dont think it makes sense adding validation tab if the account has no staking feature

@brancoder brancoder requested a review from msarcev February 26, 2024 14:01
@begonaalvarezd begonaalvarezd merged commit 2966225 into feat/add-nft-tab Feb 26, 2024
4 of 6 checks passed
@begonaalvarezd begonaalvarezd deleted the feat/add-validation-tab branch February 26, 2024 16:22
brancoder added a commit that referenced this pull request Feb 27, 2024
* feat: add output tab to address page

* fix: add components to display native tokens

* Remove unused code in AddressPageTabbedSections component

* remove unused interfaces

* Update address states and Add basic outputs API endpoint

* Add eslint-disable for unsafe return in novaApiService.ts

* Add Foundries tab to Accound address page

* Add Foundries tab to Accound address page (#1134)

* feat: add block issuance tab to account page

* fix: validation check in foundries endpoint

* fix: component imports

* fix: add is congestion loading

* fix: add state tab for Anchor Address

* fix: add state metadata feature

* feat: add nft section

* feat: add nft section + fix:add  metadata entries FeaturesView and Asset component

* fix: add null checks in BIC section

* fix: remove leftover nft metadata property in AddressPageTabbedSection

* Feat: Add validators tab to account address page (#1183)

* feat: add validators tab to account address page

* fix: update sdk commit

* fix: show validator tab if account has staking feature

* fix: key string

---------

Co-authored-by: Mario <[email protected]>
Co-authored-by: Begoña Álvarez de la Cruz <[email protected]>

---------

Co-authored-by: Mario <[email protected]>
Co-authored-by: Begoña Álvarez de la Cruz <[email protected]>
begonaalvarezd added a commit that referenced this pull request Mar 5, 2024
* feat: add output tab to address page

* fix: add components to display native tokens

* Remove unused code in AddressPageTabbedSections component

* remove unused interfaces

* Update address states and Add basic outputs API endpoint

* Add eslint-disable for unsafe return in novaApiService.ts

* Add Foundries tab to Accound address page

* Add Foundries tab to Accound address page (#1134)

* feat: add block issuance tab to account page

* fix: validation check in foundries endpoint

* fix: component imports

* fix: add is congestion loading

* fix: add state tab for Anchor Address

* fix: add state metadata feature

* feat: add nft section

* feat: add nft section + fix:add  metadata entries FeaturesView and Asset component

* fix: add null checks in BIC section

* fix: remove leftover nft metadata property in AddressPageTabbedSection

* feat: add validators tab to account address page

* fix: update sdk commit

* Add delegation section

* fix: show validator tab if account has staking feature

* Add fetch delegation ids endpoint

* fix: key string

* Add hook to fetch delegation outputs

* Show delegation outputs total amount

* Feat: Add validators tab to account address page (#1183)

* feat: add validators tab to account address page

* fix: update sdk commit

* fix: show validator tab if account has staking feature

* fix: key string

---------

Co-authored-by: Mario <[email protected]>
Co-authored-by: Begoña Álvarez de la Cruz <[email protected]>

* add delegation output rewards to Delegation Section

* fix: sdk changes to address calculations + styling fix

* fix: delegation response types

* fix: lint errors

---------

Co-authored-by: Mario <[email protected]>
Co-authored-by: Begoña Álvarez de la Cruz <[email protected]>
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.

[Task]: Add validation tab to account address page
3 participants