Skip to content

Conversation

nidhi-singh02
Copy link
Contributor

@nidhi-singh02 nidhi-singh02 commented Sep 10, 2025

Summary

Add Home Page to developer hub with same content from the documentation

Rationale

How has this been tested?

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

Copy link

vercel bot commented Sep 10, 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 Sep 10, 2025 11:28am
developer-hub Ready Ready Preview Comment Sep 10, 2025 11:28am
5 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
api-reference Skipped Skipped Sep 10, 2025 11:28am
entropy-explorer Skipped Skipped Sep 10, 2025 11:28am
insights Skipped Skipped Sep 10, 2025 11:28am
proposals Skipped Skipped Sep 10, 2025 11:28am
staking Skipped Skipped Sep 10, 2025 11:28am

@nidhi-singh02 nidhi-singh02 self-assigned this Sep 10, 2025
Comment on lines +167 to +170
<div
className={[styles.productBadge, styles.lazerBadge].join(
" ",
)}
Copy link
Contributor

Choose a reason for hiding this comment

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

Comment on lines +275 to +318
<section className={styles.quickStartSection}>
<h2 className={styles.sectionHeading}>Quick Start</h2>
<p className={styles.sectionDescription}>
Get started with Pyth Network by exploring our comprehensive
documentation:
</p>
<ul className={styles.quickStartList}>
<li>
<strong>
<Link
href="/pyth-core/getting-started"
className={styles.link}
>
Price Feeds Getting Started
</Link>
</strong>{" "}
- Integrate real-time price data
</li>
<li>
<strong>
<Link href="/lazer" className={styles.link}>
Lazer Documentation
</Link>
</strong>{" "}
- High-performance price feeds
</li>
<li>
<strong>
<Link href="/express-relay" className={styles.link}>
Express Relay Integration
</Link>
</strong>{" "}
- Better orderflow mechanism to eliminate MEV
</li>
<li>
<strong>
<Link href="/entropy" className={styles.link}>
Entropy Implementation
</Link>
</strong>{" "}
- Secure randomness generation
</li>
</ul>
</section>
Copy link
Contributor

Choose a reason for hiding this comment

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

the same structure is basically used multiples times, maybe extract it into a separate component and just reuse the component?

eg:

type ProductCardProps = {
  badge: string;
  badgeClass: string;
  icon: ReactNode;
  title: string;
  description: string;
  features: { icon: ReactNode; text: string }[];
  link: string;
  cta: string;
};

const ProductCard = ({
  badge,
  badgeClass,
  icon,
  title,
  description,
  features,
  link,
  cta,
}: ProductCardProps) => (
  <div className={styles.productCard}>
.....

<h3 className={styles.sectionTitle}>Additional Information</h3>
<ul className={styles.navList}>
<li>
<Link href="/pyth-token" className={styles.navItem}>
Copy link
Contributor

Choose a reason for hiding this comment

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

you're prob aware, but these links are broken

Comment on lines +105 to +110

@media (width <= 1024px) {
max-width: 100%;
padding: theme.spacing(6) theme.spacing(4);
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Comment on lines +226 to +231
.productTitle {
@include theme.text("lg", "semibold");

color: theme.color("heading");
margin-bottom: theme.spacing(3);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

I would also include the @include theme.h3; and for the other semantic elements

@nidhi-singh02
Copy link
Contributor Author

As discussed with @aditya520, moving it to draft as there will be a new home page design and navigation may change as well.

@nidhi-singh02 nidhi-singh02 marked this pull request as draft September 10, 2025 14:33
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.

2 participants