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

Memory processing cases that result in a bad User experience #868

Closed
4 tasks done
beastoin opened this issue Sep 17, 2024 · 6 comments
Closed
4 tasks done

Memory processing cases that result in a bad User experience #868

beastoin opened this issue Sep 17, 2024 · 6 comments
Assignees

Comments

@beastoin
Copy link
Collaborator

beastoin commented Sep 17, 2024

Edge cases to be handled:

  • When I have a conversation in progress, if I disconnect the device or the battery's off, or similar, the current conversation disappears, Instead it should maintain there and say "Waiting for reconnect...". Probably _getMemoryHeader
  • If I have a memory in progress, and kill, open the app before the memory starts processing, I expect the conversation to still be there, so the websocket should sync and show the previous content in the "in progress". (I see after processing prev transcript + new transcript works, it's just to show that properly on the user side while in progress)
  • Once the backend starts processing the memory ("120 seconds mark"), that transcript should be moved below in the capture item, to another list item, that says "Processing" and has a circular progress indicator instead of an emoji. Once completed processing, it should highlight, a small animation + maybe a snackbar (but only trigger once, save a preference for it). New segments after "processing" started, should be in the capture item.
  • When a memory is processed, and becomes discarded, but showDiscarded is not true, we should still show them, until the app is restarted, as a user, I have a few conversations and go back to the app, and don't see anything, I right away think it failed, until I remember it might have been discarded.

Solution

  • Processing memory with new field: status(capturing, processing, done), capturing_to
  • Capture provider track capturing_to, capturing status of processing memory
  • New session at Memories page for processing memories, track processing status, bellow capturing item. Use poling for now, send new memory forcing showing to the memories session.
@beastoin beastoin self-assigned this Sep 17, 2024
@beastoin beastoin converted this from a draft issue Sep 17, 2024
@beastoin
Copy link
Collaborator Author

Let's go!

@beastoin
Copy link
Collaborator Author

Solution

  • Processing memory with new field: status(capturing, processing, done), capturing_to
  • Capture provider track capturing_to, capturing status of processing memory
  • New session at Memories page for processing memories, track processing status, bellow capturing item. Use poling for now, send new memory forcing showing to the memories session.

@josancamon19
Copy link
Contributor

Testing

@josancamon19
Copy link
Contributor

Image

Duplicate isue

@beastoin
Copy link
Collaborator Author

yup

@beastoin
Copy link
Collaborator Author

also this, any UX enhancement needed? bro @josancamon19

@beastoin beastoin closed this as completed Nov 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

2 participants