Skip to content

Clarify per-conversation attachments in the composer#1091

Open
benjaminshafii wants to merge 1 commit intodevfrom
task/issue-1064-conversation-attachments
Open

Clarify per-conversation attachments in the composer#1091
benjaminshafii wants to merge 1 commit intodevfrom
task/issue-1064-conversation-attachments

Conversation

@benjaminshafii
Copy link
Member

Summary

  • add a clearer drag-and-drop target so files feel attached to the current conversation, not the shared folder
  • label pending uploads as conversation-scoped before send
  • make the paperclip affordance explicit about attaching files to the chat

Testing

  • pnpm --filter @openwork/app typecheck
  • pnpm --filter @openwork/app build
  • Not run: Docker + Chrome MCP end-to-end verification for this branch

@vercel
Copy link

vercel bot commented Mar 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-app Ready Ready Preview, Comment Mar 21, 2026 4:54pm
openwork-den Building Building Preview Mar 21, 2026 4:54pm
openwork-den-worker-proxy Ready Ready Preview, Comment Mar 21, 2026 4:54pm
openwork-landing Ready Ready Preview, Comment, Open in v0 Mar 21, 2026 4:54pm
openwork-share Ready Ready Preview, Comment Mar 21, 2026 4:54pm

@github-actions
Copy link
Contributor

The following comment was made by an LLM, it may be inaccurate:

@benjaminshafii
Copy link
Member Author

Chrome MCP verification pass completed on localhost:5173.

Verified:

  • the composer labels attachments as conversation-scoped before send
  • the paperclip affordance now reads Attach files to this conversation
  • attaching an image shows the pending attachment chip above the composer

Screenshot:

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant