add datasets #458
add datasets #458
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the explore public knowledge page to support multiple dataset types through a tabbed interface and adds new entity schemas for a more comprehensive knowledge graph.
Key changes:
- Added new entity schemas (Dapp, Investor, FundingStage, InvestmentRound, Asset) to support broader dataset exploration
- Refactored explore-public-knowledge into a layout with child routes for different dataset types
- Implemented tabbed navigation for switching between Projects, dApps, Investment Rounds, and Asset Market
Reviewed Changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 13 comments.
Show a summary per file
| File | Description |
|---|---|
| schema.ts | Added new entity class definitions for Dapp, Investor, FundingStage, InvestmentRound, and Asset |
| routes/index.tsx | Updated start exploring link to point to projects tab |
| routes/explore-public-knowledge/projects.tsx | Extracted existing projects page with search functionality |
| routes/explore-public-knowledge/investment-rounts.tsx | New page for displaying investment rounds data |
| routes/explore-public-knowledge/index.tsx | Added redirect from base path to projects tab |
| routes/explore-public-knowledge/dapps.tsx | New page for displaying dApps with GitHub integration |
| routes/explore-public-knowledge/asset-market.tsx | New page for displaying asset market data |
| routes/explore-public-knowledge.tsx | Refactored to layout component with tabbed navigation |
| routeTree.gen.ts | Auto-generated route tree updates for new child routes |
| mapping.ts | Added entity mappings for new schema classes |
| components/navbar.tsx | Updated navigation link to point to projects tab |
| components/explore-tabs.tsx | New tabbed navigation component for dataset switching |
...ate-hypergraph/template-vite-react/src/routes/explore-public-knowledge/investment-rounts.tsx
Outdated
Show resolved
Hide resolved
| <span className="text-white font-bold text-lg">{investmentRound.name.charAt(0).toUpperCase()}</span> | ||
| </div> | ||
|
|
||
| {/* Asset name */} |
There was a problem hiding this comment.
The comment says 'Asset name' but this is displaying an investment round name, not an asset name.
| {/* Asset name */} | |
| {/* Investment round name */} |
Copilot uses AI. Check for mistakes.
| {investmentRound.name} | ||
| </h3> | ||
|
|
||
| {/* Asset ID */} |
There was a problem hiding this comment.
The comment says 'Asset ID' but this is displaying an investment round ID, not an asset ID.
| {/* Asset ID */} | |
| {/* Investment Round ID */} |
Copilot uses AI. Check for mistakes.
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No Projects Found</h3> | ||
| <p className="text-gray-500">There are currently no public projects available to explore.</p> |
There was a problem hiding this comment.
The empty state message says 'No Projects Found' but this page displays investment rounds, not projects.
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No Projects Found</h3> | |
| <p className="text-gray-500">There are currently no public projects available to explore.</p> | |
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No Investment Rounds Found</h3> | |
| <p className="text-gray-500">There are currently no public investment rounds available to explore.</p> |
Copilot uses AI. Check for mistakes.
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No Projects Found</h3> | ||
| <p className="text-gray-500">There are currently no public projects available to explore.</p> |
There was a problem hiding this comment.
The empty state description mentions 'projects' but should reference 'investment rounds' for consistency with the page content.
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No Projects Found</h3> | |
| <p className="text-gray-500">There are currently no public projects available to explore.</p> | |
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No Investment Rounds Found</h3> | |
| <p className="text-gray-500">There are currently no public investment rounds available to explore.</p> |
Copilot uses AI. Check for mistakes.
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No Projects Found</h3> | ||
| <p className="text-gray-500">There are currently no public projects available to explore.</p> |
There was a problem hiding this comment.
The empty state message says 'No Projects Found' but this page displays dApps, not projects.
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No Projects Found</h3> | |
| <p className="text-gray-500">There are currently no public projects available to explore.</p> | |
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No Dapps Found</h3> | |
| <p className="text-gray-500">There are currently no public dapps available to explore.</p> |
Copilot uses AI. Check for mistakes.
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No Projects Found</h3> | ||
| <p className="text-gray-500">There are currently no public projects available to explore.</p> |
There was a problem hiding this comment.
The empty state description mentions 'projects' but should reference 'dApps' for consistency with the page content.
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No Projects Found</h3> | |
| <p className="text-gray-500">There are currently no public projects available to explore.</p> | |
| <h3 className="text-xl font-semibold text-gray-900 mb-2">No dApps Found</h3> | |
| <p className="text-gray-500">There are currently no public dApps available to explore.</p> |
Copilot uses AI. Check for mistakes.
apps/create-hypergraph/template-vite-react/src/routes/explore-public-knowledge/asset-market.tsx
Outdated
Show resolved
Hide resolved
apps/create-hypergraph/template-vite-react/src/routes/explore-public-knowledge/asset-market.tsx
Outdated
Show resolved
Hide resolved
apps/create-hypergraph/template-vite-react/src/components/explore-tabs.tsx
Outdated
Show resolved
Hide resolved
1 participant
No description provided.