Skip to content

feat: add shareable links for Addie threads in admin#737

Merged
bokelley merged 2 commits into
mainfrom
bokelley/addie-thread-sharing
Jan 13, 2026
Merged

feat: add shareable links for Addie threads in admin#737
bokelley merged 2 commits into
mainfrom
bokelley/addie-thread-sharing

Conversation

@bokelley

Copy link
Copy Markdown
Contributor

Summary

  • Add "Copy Link" button to copy shareable admin URL for any thread
  • Add "Open in Slack" link for Slack threads to jump to original conversation
  • Support ?thread= URL parameter to deep-link directly to a specific thread

Security

  • Validates Slack channel ID and timestamp format to prevent XSS
  • Escapes URLs in href attributes
  • Handles null returns to avoid broken links
  • Proper error handling for clipboard API
  • URL encodes thread IDs

Test plan

  • Navigate to Admin Addie, open a thread, verify "Copy Link" button copies correct URL
  • Open copied URL in new tab, verify it opens the correct thread
  • For Slack threads, verify "Open in Slack" button opens the original thread
  • Verify "Open in Slack" doesn't appear for web/a2a threads

🤖 Generated with Claude Code

bokelley and others added 2 commits January 12, 2026 13:16
- Add "Copy Link" button to copy shareable admin URL for any thread
- Add "Open in Slack" link for Slack threads to jump to original conversation
- Support ?thread= URL parameter to deep-link directly to a specific thread

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Validate Slack channel ID and timestamp format to prevent XSS
- Handle null return from getSlackPermalink to avoid broken links
- Add proper error handling for clipboard API
- URL encode thread ID in shareable links

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@bokelley bokelley merged commit 29318fe into main Jan 13, 2026
6 checks passed
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