Skip to content

docs: align dashboard page with Issue #3 — Vite + react-ts decision#7

Merged
aolieman merged 5 commits intomainfrom
dashboard
Mar 3, 2026
Merged

docs: align dashboard page with Issue #3 — Vite + react-ts decision#7
aolieman merged 5 commits intomainfrom
dashboard

Conversation

@KoxyG
Copy link
Member

@KoxyG KoxyG commented Feb 26, 2026

Updates the PG Atlas Dashboard doc to match the decision from Issue #3.

Changes:

  • Replaced the previous "Implementation Options" section (Streamlit, Dash, React/Next.js) with a "Technology Decision" section that records the chosen approach.
  • Decision: Option C — React/Next.js (or Vite) for the dashboard. No Panel prototype in the chosen path.
  • Rationale: TypeScript SDK dogfooding, contributor accessibility (React/TS ecosystem), and a scoped v0 (leaderboard + basic PG detail pages; graph explorer deferred to v1).
  • Ownership: KoxyG — Next.js or Vite, coordinating with the team to finalize features.
  • Trimmed "Open Questions" to items still open (graph lib, hosting, analytics, mockups).

Resolves the doc drift noted in #3 (dashboard page now consistent with the Build submission and issue resolution).

@KoxyG KoxyG requested a review from aolieman February 26, 2026 11:31
Copy link
Member

@aolieman aolieman left a comment

Choose a reason for hiding this comment

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

Very clear decision, thanks. It's okay to leave Panel out of this page since it's only part of our dev team process, and not a component that is intended to go into production. This way, we have a clearer separation of concerns, and the Dashboard page matches the component repo boundaries.

There is only one disagreement to flag: whether we need the sub-graph explorer on Project detail pages as an interactive validation instrument of the criticality score, for voters and PG maintainers.

Most of the feedback I left, I also intend to address in an upcoming commit to this PR.

@aolieman aolieman self-requested a review February 26, 2026 13:58
@aolieman aolieman changed the title docs: align dashboard page with Issue #3 — React/Next.js (or Vite) decision docs: align dashboard page with Issue #3 — Vite + react-ts decision Feb 27, 2026
Copy link
Member

@aolieman aolieman left a comment

Choose a reason for hiding this comment

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

Looks good to me! If you're also happy @KoxyG, please squash merge this PR.

@KoxyG
Copy link
Member Author

KoxyG commented Feb 27, 2026

Looks good to me! If you're also happy @KoxyG, please squash merge this PR.

Merging is blocked from my end

@aolieman
Copy link
Member

aolieman commented Mar 3, 2026

Merging is blocked from my end

Sorry, that was my bad. I didn't see one of the comments was marked as unresolved, nor did I know this would block the merge. Please go ahead and approve & squash merge.

@KoxyG KoxyG self-assigned this Mar 3, 2026
Copy link
Member Author

@KoxyG KoxyG left a comment

Choose a reason for hiding this comment

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

good to go

@aolieman aolieman merged commit 4b85e7d into main Mar 3, 2026
1 check passed
@aolieman aolieman deleted the dashboard branch March 3, 2026 21:58
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