Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

feat: add threads sidebar interactivity #132

Open
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

tobySolutions
Copy link
Member

This PR fixes #125, although the API has not been connected yet. Things I was able to complete are:

  1. Added a loading skeleton for when fetching.
  2. Displayed thread titles that truncates when it exceeds a wrap.
  3. Added in hover states, selected states, etc for UX feedback (anything you can think of, please refer to actual assistants playground)

Please review accordingly.

Copy link

vercel bot commented Jun 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
akeru-akeru-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 23, 2024 7:50pm
akeru-app-akeru ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 23, 2024 7:50pm
akeru-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 23, 2024 7:50pm

@multipletwigs
Copy link
Collaborator

multipletwigs commented Jun 11, 2024

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 akeru-server package locally! You can use run redis through docker to set up our db! There's also a new script called generateHumanUser that generates a few fake threads and assistants, and prints out the Bearer <token> that would be used by your user. It is currently stored as an env variable and ideally, when we have an auth system, we should be storing it into our session storage, or something along with appropriate timeout sessions.

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 rebase or merge?

@tobySolutions
Copy link
Member Author

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 akeru-server package locally! You can use run redis through docker to set up our db! There's also a new script called generateHumanUser that generates a few fake threads and assistants, and prints out the Bearer <token> that would be used by your user. It is currently stored as an env variable and ideally, when we have an auth system, we should be storing it into our session storage, or something along with appropriate timeout sessions.

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 rebase or merge?

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.

@tobySolutions
Copy link
Member Author

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 akeru-server package locally! You can use run redis through docker to set up our db! There's also a new script called generateHumanUser that generates a few fake threads and assistants, and prints out the Bearer <token> that would be used by your user. It is currently stored as an env variable and ideally, when we have an auth system, we should be storing it into our session storage, or something along with appropriate timeout sessions.

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 rebase or merge?

I will ensure to stick to rebasing

threads.get(
"/thread",
async ({ bearer, set }) => {
console.info("all threads baby");
Copy link
Collaborator

Choose a reason for hiding this comment

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

do remove console!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Akeru Web App] Displaying all existing threads for a user
2 participants