Skip to content

Add Developer Hub to monorepo #2672

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

Merged
merged 15 commits into from
May 21, 2025
Merged

Add Developer Hub to monorepo #2672

merged 15 commits into from
May 21, 2025

Conversation

aaronbassett
Copy link
Contributor

@aaronbassett aaronbassett commented May 9, 2025

Summary

Minimal Developer Hub boilerplate with a basic content structure so we can begin Vercel deployments.

@aaronbassett aaronbassett requested a review from a team as a code owner May 9, 2025 19:01
Copy link

vercel bot commented May 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api-reference ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 21, 2025 0:17am
component-library ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 21, 2025 0:17am
entropy-debugger ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 21, 2025 0:17am
entropy-explorer ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 21, 2025 0:17am
insights ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 21, 2025 0:17am
proposals ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 21, 2025 0:17am
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 21, 2025 0:17am

Copy link
Collaborator

@cprussin cprussin left a comment

Choose a reason for hiding this comment

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

Overall looking great! Some minor changes but nothing big and then I think this will be good to merge

Copy link
Collaborator

@cprussin cprussin left a comment

Choose a reason for hiding this comment

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

Looking very good! A few minor points and notes and then I think we'll be good to ship it

@@ -0,0 +1,77 @@
import createBundleAnalyzer from "@next/bundle-analyzer";
Copy link
Collaborator

@cprussin cprussin May 17, 2025

Choose a reason for hiding this comment

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

Nice! Note that I'm working on moving other apps to using turbopack; I'm not sure how well this will work with turbopack, but in the meantime I think it would be great to roll it out across all our apps (obviously no need to do it in this PR but I'll be curious to see how you use the bundle analyzer and see if we want to leverage it elsewhere, so please let me know your thoughts when you get some more usage)

var(--color-steel-50)
);
--color-fd-ring: light-dark(var(--color-violet-600), var(--color-violet-400));
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Very nice! I suggest trying to use theme.scss to drive this file if possible -- in particular I think the whole color gradient can just use theme.pallette-color, and I think all the color tokens can be defined in terms of theme.color values right?

e.g. I would imagine this file could look something like this:

@use "@pythnetwork/component-library/theme";

:root {
  --color-slate-50: theme.pallette-color("slate", 50);
  ...

  --color-fd-background: theme.color("background", "primary");
}

One thing to note is that once the new website gets rolled out, we will do a pass at adjusting the design system color tokens a bit to match with the new branding. If you implement in terms of all these tokes, it will mean less that we have to worry about manually syncing.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't want to spend any time on this part as I'm hoping @partyparrotgreg will replace it with the correct color mapping.

Update start:dev and start:prod ports so they do not clash with
Insights or other local apps.
@@ -25,3 +25,4 @@ __pycache__
.turbo/
.cursorrules
.corepack
justfile
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hmm what is this? Is it intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, justfile is like a Makefile (but with a simplified syntax) it's a local command runner I use in terminal which probably isn't helpful to commit

Copy link
Collaborator

Choose a reason for hiding this comment

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

totally reasonable, feel free to commit files like this if you think they may be useful for others, for instance I've committed my nix files (flake.nix etc) -- maybe in a separate commit though

@vercel vercel bot temporarily deployed to Preview – insights May 20, 2025 22:56 Inactive
@vercel vercel bot temporarily deployed to Preview – staking May 20, 2025 22:56 Inactive
@vercel vercel bot temporarily deployed to Preview – api-reference May 20, 2025 22:56 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-debugger May 20, 2025 22:56 Inactive
@vercel vercel bot temporarily deployed to Preview – proposals May 20, 2025 22:56 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-explorer May 20, 2025 22:56 Inactive
@vercel vercel bot temporarily deployed to Preview – staking May 20, 2025 23:04 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-explorer May 20, 2025 23:04 Inactive
@vercel vercel bot temporarily deployed to Preview – insights May 20, 2025 23:04 Inactive
@vercel vercel bot temporarily deployed to Preview – api-reference May 20, 2025 23:04 Inactive
@vercel vercel bot temporarily deployed to Preview – proposals May 20, 2025 23:04 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-debugger May 20, 2025 23:04 Inactive
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