Disable leaderboard

autonomys / astral

Home of our Block Explorer

https://explorer.subspace.network

11 stars 9 forks source link

Disable leaderboard #906

Closed marc-aurele-besner closed 1 week ago

marc-aurele-besner commented 1 week ago

User description

PR Type

enhancement

Description

Enhanced the leaderboard route by adding a networks property.
Restricted the leaderboard route to NetworkId.GEMINI_3H.

Changes walkthrough 📝

Relevant files

Enhancement

routes.ts

Add network restriction to leaderboard route

explorer/src/constants/routes.ts

Added networks property to the leaderboard route.

Set networks to include NetworkId.GEMINI_3H.

+1/-0

Additional files (token-limit)

graphql.ts

...

explorer/gql/graphql.ts ...

+12453/-10796

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

netlify[bot] commented 1 week ago

Deploy Preview for dev-astral ready!

Name	Link
Latest commit	80f309b12dafa13062e74c81881289752fb987ca
Latest deploy log	https://app.netlify.com/sites/dev-astral/deploys/672a97ee5e8ecf00085bdded
Deploy Preview	https://deploy-preview-906--dev-astral.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 configuration.

github-actions[bot] commented 1 week ago

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review

Hardcoded Value
The addition of `networks` with a hardcoded value `[NetworkId.GEMINI_3H]` might limit flexibility. Consider making this configurable or validating the necessity of this hardcoding.