-
Notifications
You must be signed in to change notification settings - Fork 10
feat: Artist TikTok Connections via Composio #1501
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: test
Are you sure you want to change the base?
Conversation
- Add useArtistConnectors hook for managing artist-specific connectors (TikTok) - Follows same patterns as existing useConnectors hook - Supports fetch, authorize, disconnect functions with Bearer token auth - Uses NEW_API_BASE_URL for API calls Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Add @radix-ui/react-tabs dependency - Create components/ui/tabs.tsx following shadcn pattern - Refactor Settings.tsx to use tabs for UPDATE mode - Add General and Connections tabs with full accessibility - Create ConnectionsTab placeholder component - Preserve existing settings functionality under General tab - CREATE mode shows original form without tabs Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Implement ConnectionsTab with useArtistConnectors hook integration - Add YouTubeCard wrapper for existing StandaloneYoutubeComponent - Add TikTokCard component with connect/disconnect functionality - Show loading state with skeletons while fetching connectors - Display YouTube and TikTok connector cards in a grid layout Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Add connecting state with spinner - Add dropdown menu with Reconnect and Disconnect options - Add confirmation dialog before disconnecting - Update props to use Promise return types for async operations - Follow ConnectorConnectedMenu and DeleteFileDialog patterns Co-Authored-By: Claude Opus 4.5 <[email protected]>
Add callback handling for artist TikTok OAuth flow: - Add complete() function to useArtistConnectors hook - Create useArtistConnectorCallback hook to detect callback params - Create ArtistConnectorSuccessBanner component for success toast - Integrate callback handler in Chat component Co-Authored-By: Claude Opus 4.5 <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 33adfe9f88
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
hooks/useArtistConnectorCallback.ts
Outdated
| // Clear URL params after handling | ||
| window.history.replaceState({}, "", "/chat"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Preserve roomId when clearing OAuth query params
The OAuth callback handler replaces the URL with a hardcoded /chat, which strips the current roomId. In this app the chat route is /chat/[roomId], so after a successful connection the browser URL becomes invalid and a refresh/share will 404 or lose the active room context. Use the current pathname (e.g., window.location.pathname) or a router replace that only removes the query string so the roomId is preserved.
Useful? React with 👍 / 👎.
- Remove complete() function from useArtistConnectors hook - Simplify useArtistConnectorCallback - no API call needed - Composio stores connections automatically on OAuth callback
Summary
Stories Implemented
Test plan
🤖 Generated with Claude Code