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

feat: onboarding revamp #852

Draft
wants to merge 23 commits into
base: main
Choose a base branch
from

Conversation

oliverqx
Copy link
Contributor

@oliverqx oliverqx commented Dec 6, 2024

/claim #810

description

Essentially once im done the onboarding flow would have no bugs, be updated to include some new steps and reorganized for a better ux, but most importantly it would be really easy to extend it or reorganize it

related issue: #810

Copy link

vercel bot commented Dec 6, 2024

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

Name Status Preview Comments Updated (UTC)
screenpipe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 11, 2024 8:41pm

@oliverqx
Copy link
Contributor Author

oliverqx commented Dec 6, 2024

i just realize that this step is not persistent. Once you select the option, you either configure api keys or you dont, but the choice is not saved.

Screenshot 2024-12-06 at 11 02 33

I was thinking it should be saved and so status could be updated to show:

  1. API key status if ai-enhanced search was chosen
  2. A button to setup api if user wants to enable ai-enhanced search

@louis030195
Copy link
Collaborator

louis030195 commented Dec 9, 2024

@oliverqx whats the status? want to merge soon - like today or tomorrow

@oliverqx
Copy link
Contributor Author

@louis030195 sorry was keeping track of the issue thread, didnt see this till today.

I think i can be done today. UI is done, will push in a bit. Today i'll work on hashing out state machine details

- svg
- element
- llm models status
- permissions status
- screenpipe logo
- system apps
- system relations
- system terminals
@louis030195
Copy link
Collaborator

any update

@oliverqx
Copy link
Contributor Author

@louis030195 sorry been afk this past week. will push state machine tonight

@louis030195
Copy link
Collaborator

any news?

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

Successfully merging this pull request may close these issues.

2 participants