feat: add quests banner and bump sdk to 0.2.3#66
Conversation
|
🚀 Deployed on https://69cba7ed299c730ef9a6e5a0--davinci-ui.netlify.app |
There was a problem hiding this comment.
Pull request overview
This pull request adds a new announcement banner component to promote the DAVINCI Quests application and bumps the SDK version to 0.2.3. The changes include a new reusable QuestsAnnouncementBanner component that displays promotional content with a call-to-action button linking to the quests platform, integrated into the home page below the main header.
Changes:
- Add new
QuestsAnnouncementBannercomponent with responsive design and external link to the quests platform - Integrate the banner into the HomePage below the page header
- Bump
@vocdoni/davinci-sdkdependency from 0.2.2 to 0.2.3
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/components/quests-announcement-banner.tsx | New announcement banner component with responsive layout, external link, and consistent styling |
| src/pages/HomePage.tsx | Imports and renders the new QuestsAnnouncementBanner component in the page layout |
| package.json | Updates SDK version from ~0.2.2 to ~0.2.3 |
| pnpm-lock.yaml | Updates lock file to reflect SDK version change and dependency adjustments |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| import { ExternalLink } from 'lucide-react' | ||
| import { Button } from '~components/ui/button' | ||
| import { Card, CardContent, CardTitle } from '~components/ui/card' | ||
| import { cn } from '~lib/utils' |
There was a problem hiding this comment.
The component uses client-side UI components (Card, Button, CardTitle from '~components/ui') which all have the 'use client' directive. For consistency with similar components in the codebase (like DavinciInfoCard), and to ensure proper Next.js app router behavior, add the 'use client' directive at the top of this file.
No description provided.