discover section #808
discover section #808
Conversation
Add wallet connect functionality.
🦋 Changeset detectedLatest commit: 5c27ca9 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@jkbktl is attempting to deploy a commit to the Status Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull Request Overview
This PR implements a new "Discover" section that displays curated dApps and services built on Status Network. The changes introduce a comprehensive app discovery interface with filtering, categorization, and enhanced visual design.
- Adds data structures and UI components for app discovery with static data
- Implements filtering by popularity, new apps, and categories
- Creates enhanced app card components with cover images and social links
Reviewed Changes
Copilot reviewed 10 out of 31 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/status-network/src/components/button/index.tsx | Adds new 'grey' button variant with hover states |
| apps/hub/tailwind.config.ts | Adds skeleton animation keyframes and configuration |
| apps/hub/src/types/app.ts | Defines TypeScript interfaces for App, Category, and Tab |
| apps/hub/src/data/featured-apps.ts | Static data for featured apps display |
| apps/hub/src/data/apps.ts | Static data for all apps with filtering properties |
| apps/hub/src/constants/tabs.ts | Tab configuration for Popular/New/All filters |
| apps/hub/src/constants/categories.ts | Category definitions with emoji labels |
| apps/hub/src/app/discover/page.tsx | Main discover page with filtering and layout |
| apps/hub/src/app/_components/app-card.tsx | Enhanced app card component with images and skeleton loader |
| .changeset/friendly-cats-draw.md | Changeset documentation for the feature |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <[email protected]>
jkbktl
requested review from
a team,
JulesFiliot,
felicio,
jinhojang6 and
nonuwa-oss
jkbktl
force-pushed
the
feat/wallet-connect
branch
from
99fde63 to
f2e7acf
Compare
github-project-automation
bot
moved this from In Progress
to Done
in Web & User Interfaces
| } | ||
|
|
||
| const ConnectButton = (props: Props) => { | ||
| const { size = '32', label = 'Connect wallet' } = props |
There was a problem hiding this comment.
I would add two constants like DEFAULT_SIZE, DEFAULT_LABEL if we are picky
github-project-automation
bot
moved this from In Progress
to Waiting
in Web & User Interfaces
There was a problem hiding this comment.
Sorry @JulesFiliot, I've clicked the Re-request review button by mistake.
The PR looks good to me if i18n doesn’t matter at this point, the merge conflict will be resolved, and Jules' suggestion will be applied.
5 participants
closes #776