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

Persona using omi data and twitter #1775

Merged
merged 20 commits into from
Feb 19, 2025
Merged

Conversation

mdmohsin7
Copy link
Collaborator

@mdmohsin7 mdmohsin7 commented Feb 9, 2025

Part of #1535

  • Allow users to create personas from their omi data in one click
  • Shareable link to chat with the persona on the web (only if public)
  • Update persona whenever a new conversation is added
  • Chat with the persona within the app (files chatting not supported)
  • Connect twitter to existing and new personas
  • New personas creation flow without auth

@kodjima33
Copy link
Collaborator

kodjima33 commented Feb 11, 2025

hey @mdmohsin7 this is an amazing start - you can lock the bounty on you!

need few changes though, firstly few questions

  1. you are saying it's compatible with existing apps but I don't see a way to create standard apps anymore - where do you want it to be in the flow?
  2. facts in the app => how will they be auto-populated when my omi gets added new facts? currently i see the structure as a chat prompt, how do you want to make it?
  3. new facts looks better but it's not how it should be. Can you check Bee's facts system?

@mdmohsin7
Copy link
Collaborator Author

Sharing the answers here as well (which were answered on the meet)

  1. I commented that out for the video
  2. Every time new facts are added, the prompt is updated to reflect the new facts in a condensed structure. This works exactly like the existing personas on the web but uses tweets instead of facts.

@mdmohsin7
Copy link
Collaborator Author

Not completed yet, need to handle a few cases and there are some improvements that can be done.

@mdmohsin7
Copy link
Collaborator Author

mdmohsin7 commented Feb 16, 2025

Image

@mdmohsin7 mdmohsin7 changed the title Persona from device [MVP] Persona using omi data Feb 17, 2025
@mdmohsin7 mdmohsin7 changed the title Persona using omi data Persona using omi data and twitter Feb 19, 2025
@mdmohsin7 mdmohsin7 changed the base branch from main to mobile_personas February 19, 2025 06:52
@beastoin
Copy link
Collaborator

0/ did you cover the flow of no device - Rishabh's work ?
1/ how many apps would be created if i create persona from omi app first, then connect to a X account ?

@mdmohsin7

@mdmohsin7
Copy link
Collaborator Author

mdmohsin7 commented Feb 19, 2025

0/ Yes sir, I've covered it
1/ Only one app will be created. The prompt will be updated to reflect both twitter and user facts and conversations

@beastoin

@beastoin
Copy link
Collaborator

beastoin commented Feb 19, 2025

0/ ✅ ok
1/ ✅ ok
2/ how about creating the X persona first ?
3/ we did not need the user doing the X OAuth, rapid api is all we need, right ?
4/ so the DMs feature is not ready in this PR, right ?

@mdmohsin7

@mdmohsin7
Copy link
Collaborator Author

2/ New users -> X persona first
Existing user -> option to connect X while creating persona with omi data
3/Yes, we are verifying if their latest tweet is "verifying omi persona" or not to make sure that they own the account
4/ Yes sir

@beastoin

@beastoin
Copy link
Collaborator

2/ i mean how many persona would be created if I create a new persona by X then connect to the omi...oops ?
3/ ✅ ok
4/ ✅ ok

@mdmohsin7

@mdmohsin7
Copy link
Collaborator Author

2/ If you create a persona with X, then you create your account with apple/google and try to create persona with omi data, then only one will be created because you are updating the existing X created persona

They can also create multiple personas with X and omi connected, so there's no limit on number of personas a user can create

@beastoin

@beastoin
Copy link
Collaborator

2/ ✅ ok, noted

i will push this one to the testflight. be ready for more feedback.

@mdmohsin7

@beastoin beastoin merged commit 598ed96 into mobile_personas Feb 19, 2025
@beastoin beastoin deleted the personas-from-device branch February 19, 2025 07:57
@beastoin
Copy link
Collaborator

pr: #1828

@beastoin beastoin mentioned this pull request Feb 19, 2025
4 tasks
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.

3 participants