Skip to content
This repository was archived by the owner on Aug 5, 2025. It is now read-only.

feat(react-chat): add DashboardCard component #532

Closed
devin-ai-integration wants to merge 3 commits into from
Closed

feat(react-chat): add DashboardCard component #532

devin-ai-integration wants to merge 3 commits into from

Conversation

@devin-ai-integration
Copy link

@devin-ai-integration devin-ai-integration bot commented Jan 16, 2025 •
edited
Loading

feat(react-chat): add AgentCard component

Added a new AgentCard component with the following features:

  • 6px border radius
  • Specified box shadows:
    • 0px 1px 3px 1px #161A1E03
    • 0px 4px 8px -6px #161A1E14
    • 0px 1px 5px -4px #161A1E14
    • 0px 0px 0px 1px #161A1E0A
    • 0px 1px 0px 0px #161A1E05
  • Generic and flexible interface accepting arbitrary children
  • Storybook stories demonstrating simple and complex content usage

Implementation Details:

// packages/react-chat/src/components/AgentCard/styled.ts
export const Container = styled(tag('div'), {
  borderRadius: '6px',
  boxShadow: `
    0px 1px 3px 1px #161A1E03,
    0px 4px 8px -6px #161A1E14,
    0px 1px 5px -4px #161A1E14,
    0px 0px 0px 1px #161A1E0A,
    0px 1px 0px 0px #161A1E05
  `,
  display: 'flex',
  flexDirection: 'column',
  boxSizing: 'border-box',
  overflow: 'hidden',
  backgroundColor: '$white',
});

Link to Devin run: https://app.devin.ai/sessions/bfec985b5ee34e8297a4a75c0704e7c4

@devin-ai-integration @mikaalnaik
feat(react-chat): add DashboardCard component
569b624 
- Add new DashboardCard component with 6px border radius and specified box shadows
- Create story file demonstrating simple and complex content usage
- Add DASHBOARD_CARD to ClassName enum

Co-Authored-By: mikaal.naik@voiceflow.com <mikaal.naik@voiceflow.com>
@devin-ai-integration
Copy link
Author

devin-ai-integration bot commented Jan 16, 2025

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@graphite-app
Copy link

graphite-app bot commented Jan 16, 2025

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 16, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
2.2% Coverage on New Code (required ≥ 80%)
0.0% Condition Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

devin-ai-integration bot and others added 2 commits January 16, 2025 21:42
@devin-ai-integration @mikaalnaik
test(react-chat): add comprehensive tests for DashboardCard component
4351adc 
Co-Authored-By: mikaal.naik@voiceflow.com <mikaal.naik@voiceflow.com>
@devin-ai-integration @mikaalnaik
refactor(react-chat): rename DashboardCard to AgentCard
75dcd2f 
Co-Authored-By: mikaal.naik@voiceflow.com <mikaal.naik@voiceflow.com>
graphite-app[bot]
graphite-app bot reviewed Jan 16, 2025
View reviewed changes
packages/react-chat/src/components/AgentCard/AgentCard.story.tsx
};

/**
* @see {@link https://voiceflow.github.io/react-chat/?path=/story/components-dashboard-card--simple}
Copy link

@graphite-app graphite-app bot Jan 16, 2025

Choose a reason for hiding this comment

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

The Storybook documentation link contains an incorrect path - it points to components-dashboard-card but should point to components-agent-card to match the component name. The correct link should be:

https://voiceflow.github.io/react-chat/?path=/story/components-agent-card--simple

Spotted by Graphite Reviewer

Is this helpful? React 👍 or 👎 to let us know.

@effervescentia effervescentia force-pushed the devin/1737062431-dashboard-card branch from bae3be3 to 75dcd2f Compare January 17, 2025 17:39
@devin-ai-integration
Copy link
Author

devin-ai-integration bot commented Jan 25, 2025

Closing due to inactivity.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@graphite-app graphite-app[bot] graphite-app[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant