-
Notifications
You must be signed in to change notification settings - Fork 425
[feature] Add user onboarding #2986
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
base: release/v0.69.3
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…into feature/user-onbroading-to-app
…re/user-onbroading-to-app
mmabrouk
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added some comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand this change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated this file and evaluator_manager.py to add a default LLM-as-a-judge evaluator. I have done that using AI
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand this change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same answer as the above question
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand this change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First check the below asnwer for better context. Since we have added a patch directory, we need to add that on dockerfile as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what does this do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, the library we are using, nextstepjs for targeted UI onboarding tour, the library has some default that's why we are using it. But also, it has some limitations, like the library didn'tprovided any ways to retry to find an element if it's not available on first look, it's a patch for the library add the ability to retry if an element is not found on first try.
I know it's not a really good way to do this. It's non issue on their repo as well, they have an issue for this. Until they fix that issue. It's a workaround for us.
I mean I could raise a PR on there repo but didn't find enough time to do that. also forgot about that
…m and previewEvalTypeAtom for improved evaluation flow. Update related atoms and ensure consistent usage across onboarding steps and URL resolution.
…stency. Update type definitions to use interfaces instead of types where applicable, and streamline imports across various onboarding files. Enhance the onboarding steps and state management for better clarity and maintainability.
What's new??
We have introduced the Onboarding feature on our application to onboard any new or existing user in a specific feature. The current implementation of the onboarding is very basic, but we have created the foundation to extend it and make it customizable based on user data.
Where do we have onboarding as for now ??