#5889 hiding stats

[prachi00]

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.

👇 __ Let's make a quick check before the contribution.

PR Type

• Bugfix
• Feature
• Refactoring

Context

• Closes Hide Stats #5889
• Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

• My contribution builds clean without any errors or warnings
• I've merged recent default branch -- main and I've no conflicts
• I've tried to respect high code quality standards
• I've didn't break any original functionality

Optional

• I've tested it at </ksm/collection>
• I've tested PR on mobile
• I've written unit tests 🧪
• I've found edge cases

Had issue bounty label?

• Fill up your KSM address:
  Payout

Community participation

• Are you at KodaDot Discord?

Screenshot 📸

• My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖 Generated by Copilot at eb8cd9a

Removed StatsDropdown from navbar and cleaned up unused code. This change makes the navbar faster and less cluttered.

🤖 Generated by Copilot at eb8cd9a

Sing, O Muse, of the swift and skillful coder
Who from the navbar StatsDropdown removed
To make the page more swift and less encumbered
And please the users with a smoother groove.

[kodabot]

SUCCESS @prachi00 PR for issue #5889 which is assigned to you. Please wait for review and don't hesitate to grab another issue in the meantime!

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	c40454f
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/645dd364ffb67500080febaf
😎 Deploy Preview	https://deploy-preview-5894--koda-canary.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[reviewpad]

AI-Generated Summary: This pull request removes the StatsDropdown component from Navbar.vue and its associated import statement. This results in the stats dropdown no longer being displayed in the navigation bar.

[preschian]

adjust e2e test also please

nft-gallery/tests/cypress/support/commands.ts

Line 33 in 2a950ae

cy.get('[data-cy="stats"]').should('be.visible')

. otherwise lgtm

[roiLeo]

  2 failing

  1) landingRmrk
       loadLanding:
     AssertionError: Timed out retrying after 10000ms: Expected to find element: `[data-cy="spotlight"]`, but never found it.
      at Context.eval (webpack:///./cypress/support/commands.ts:34:5)

  2) landingBsx
       loadLanding:
     AssertionError: Timed out retrying after 10000ms: Expected to find element: `[data-cy="global-offers"]`, but never found it.
      at Context.eval (webpack:///./cypress/support/commands.ts:56:5)

Comment code instead of remove it?

[preschian]

@prachi00 almost, and fix deepscan

  2 failing

  1) landingRmrk
       loadLanding:
     AssertionError: Timed out retrying after 10000ms: Expected to find element: `[data-cy="series-insight"]`, but never found it.
      at Context.eval (webpack:///./cypress/support/commands.ts:34:5)

  2) landingBsx
       loadLanding:
     AssertionError: Timed out retrying after 10000ms: Expected to find element: `[data-cy="offers-stats"]`, but never found it.
      at Context.eval (webpack:///./cypress/support/commands.ts:53:5)

[prachi00]

@prachi00 almost, and fix deepscan

  2 failing

  1) landingRmrk
       loadLanding:
     AssertionError: Timed out retrying after 10000ms: Expected to find element: `[data-cy="series-insight"]`, but never found it.
      at Context.eval (webpack:///./cypress/support/commands.ts:34:5)

  2) landingBsx
       loadLanding:
     AssertionError: Timed out retrying after 10000ms: Expected to find element: `[data-cy="offers-stats"]`, but never found it.
      at Context.eval (webpack:///./cypress/support/commands.ts:53:5)

fixed

[preschian]

fixed

still error. could you please test in your local first and then commit 😅

[prachi00]

fixed

still error. could you please test in your local first and then commit 😅

yes would you mind confirming the correct e2e command with me, I run test:e2e this on my local but it never finds these errors it seems

EDIT: resolved now

[codeclimate]

Code Climate has analyzed commit c40454f and detected 0 issues on this pull request.

View more on Code Climate.

[preschian]

yes would you mind confirming the correct e2e command with me, I run test:e2e this on my local but it never finds these errors it seems

aahh i see, for me usually run test:e2e-gui and test specific test case

[vikiival]

pay 10 usd

[yangwao]

😍 Perfect, I’ve sent the payout
💵 $10 @ 24.69 USD/KSM ~ 0.405 $KSM
🧗 EzGc4s9PgCPx1YnF3fqzhLzVHpHMTL4LWPScwpDrR8JKgSU
🔗 0xbad30f2dd5da36dff8e527c6e21ade28d61567444d629d6de5f0b9b96e344d17

🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues