Skip to content

chore(design-system): add cleanup sb comments and collaboration [AR-78169] - #682

Merged
vpolessky-dn merged 5 commits into
drivenets:mainfrom
vpolessky-dn:chore/AR-78169-clean-up-sb-comments-and-collaboration
Sep 7, 2026
Merged

chore(design-system): add cleanup sb comments and collaboration [AR-78169]#682
vpolessky-dn merged 5 commits into
drivenets:mainfrom
vpolessky-dn:chore/AR-78169-clean-up-sb-comments-and-collaboration

Conversation

@vpolessky-dn

Copy link
Copy Markdown
Collaborator

No description provided.

@netlify

netlify Bot commented Sep 3, 2026

Copy link
Copy Markdown

Deploy Preview for drivenets-design-system ready!

Name Link
🔨 Latest commit 5aec289
🔍 Latest deploy log https://app.netlify.com/projects/drivenets-design-system/deploys/6a9ea4dc60a5b10007fd8265
😎 Deploy Preview https://deploy-preview-682--drivenets-design-system.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

mmurawski-dn
mmurawski-dn previously approved these changes Sep 4, 2026
…n-up-sb-comments-and-collaboration

# Conflicts:
#	packages/design-system/src/components/ds-bulk-actions/ds-bulk-actions.stories.tsx

### MCP manifest
const Default = () => <DsCommentCard
onClick={fn()}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should it be fn()?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

onClick/onResolve/onDelete are fn() (from storybook/test) in meta.args our standard for handler args and only the MCP manifest prints the literal fn() (Show code already renders () => {}), consistently across every component with handler args, so it's intentional here rather than specific to this card.

@vpolessky-dn
vpolessky-dn merged commit b2ad6af into drivenets:main Sep 7, 2026
35 of 36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants