-
Notifications
You must be signed in to change notification settings - Fork 17
feat: add threads sidebar interactivity #132
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
packages/app-akeru/src/app/(dashboard)/threads/features/thread-list/threads-list-item.tsx
Outdated
Show resolved
Hide resolved
packages/app-akeru/src/app/(dashboard)/threads/features/thread-list/threads-list.tsx
Outdated
Show resolved
Hide resolved
packages/app-akeru/src/app/(dashboard)/threads/features/thread-list/threads-list.tsx
Show resolved
Hide resolved
Hey Toby! Thanks for the PR, though I have a few comments on this! You can actually try calling the endpoint as if you're working with the actual backend by running the One other thing to note is that your commit messages seem to also include pulling and merging, i was wondering if your pulling strategy is |
Yeah, my pulling strategy is actually pulling. For some reason, it's super weird that all these commits make it here. I will investigate that. |
I will ensure to stick to rebasing |
426cb65
to
51b22cb
Compare
51b22cb
to
4c16d56
Compare
threads.get( | ||
"/thread", | ||
async ({ bearer, set }) => { | ||
console.info("all threads baby"); |
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.
do remove console!
This PR fixes #125, although the API has not been connected yet. Things I was able to complete are:
Please review accordingly.