Skip to content
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

Secretary POC #436

Merged
merged 11 commits into from
Jun 20, 2023
Merged

Secretary POC #436

merged 11 commits into from
Jun 20, 2023

Conversation

sahil-shubham
Copy link
Member

@sahil-shubham sahil-shubham commented Jun 14, 2023

In this PR

  • The conversation stack should reset on node change
  • Currently only the current conversation stack is stored, would have to old conversations as well till the user logs out.
    • Render old conversation if history exists
    • Would also have to add a scroll to bottom button that pops up when scrolling through old conversation
  • The stack is updated only after response which leads to 2 messages at once (EDIT: not that bad of a thing, if the first message fails you can just press enter again)
    • Adding new message should scroll to bottom
    • The response is in markdown, take care of it
  • Add an insert to note button
    • Would have to provide a clear conversation button as well. I cannot go about logging in and out to remove the history

Demo

secretary-demo.mov

@changeset-bot
Copy link

changeset-bot bot commented Jun 14, 2023

🦋 Changeset detected

Latest commit: cf0aba3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
mexit-webapp Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@sahil-shubham sahil-shubham marked this pull request as ready for review June 19, 2023 15:34
@sahil-shubham sahil-shubham changed the title [WIP] Secretary POC Secretary POC Jun 19, 2023
@dineshsingh1 dineshsingh1 merged commit e4ad662 into main Jun 20, 2023
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.

2 participants