Skip to content

Conversation

@fhqvst
Copy link
Contributor

@fhqvst fhqvst commented Dec 12, 2025

Summary

Some small changes to the front page of insights hub to better highlight our price feeds.

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

@fhqvst fhqvst self-assigned this Dec 12, 2025
@fhqvst fhqvst requested a review from a team as a code owner December 12, 2025 10:40
@linear
Copy link

linear bot commented Dec 12, 2025

@vercel
Copy link

vercel bot commented Dec 12, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
component-library Ready Ready Preview Comment Dec 12, 2025 10:40am
developer-hub Ready Ready Preview Comment Dec 12, 2025 10:40am
entropy-explorer Ready Ready Preview Comment Dec 12, 2025 10:40am
insights Ready Ready Preview Comment Dec 12, 2025 10:40am
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
api-reference Skipped Skipped Dec 12, 2025 10:40am
proposals Skipped Skipped Dec 12, 2025 10:40am
staking Skipped Skipped Dec 12, 2025 10:40am


&[data-hovered] {
@include theme.elevation("primary", 2);
@include theme.elevation("primary", 1);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Personally I preferred elevation 2 here...

/>
<ChartCard
header="Publishers Onboarded"
header="Publishers"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should remain "Publishers Onboarded" -- we have 2 different numbers for publishers that we highlight in different places:

  • "Publishers Onboarded" refers to the number of publishers who can publish / be permissioned
  • "Active Publishers" refers to the number of publishers who are actively permissioned in pythnet and are pushing data

We used to just have the word "Publishers" here but it was confusing because people would see different counts in different places.

Obviously I don't care about the exact terminology, but the distinction needs to exist.

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