Skip to content

Conversation

@ashrafchowdury
Copy link
Contributor

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 ??

  1. App-mangement page
  2. Playground
  3. Online evaluation
  4. Trace drawer

@vercel
Copy link

vercel bot commented Nov 17, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
agenta-documentation Ready Ready Preview, Comment Dec 17, 2025 3:15pm

@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. feature UI labels Nov 17, 2025
@junaway junaway changed the title [Feat]: user onbroading (clean-branch) [feature] Add user onboarding Nov 17, 2025
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:XXL This PR changes 1000+ lines, ignoring generated files. labels Nov 19, 2025
Copy link
Member

@mmabrouk mmabrouk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added some comments

Copy link
Member

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

Copy link
Contributor Author

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

Copy link
Member

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

Copy link
Contributor Author

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

Copy link
Member

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

Copy link
Contributor Author

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

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what does this do?

Copy link
Contributor Author

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.
@mmabrouk mmabrouk changed the base branch from main to release/v0.69.3 December 17, 2025 14:26
…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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature size:XXL This PR changes 1000+ lines, ignoring generated files. UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants