feat(ui): replace legacy product tour with onboarding v2 #14581
feat(ui): replace legacy product tour with onboarding v2 #14581
Conversation
🐋 Docker imagedocker run --pull=always --rm -it -p 8080:8080 --user=root -v /var/run/docker.sock:/var/run/docker.sock -v /tmp:/tmp ghcr.io/kestra-io/kestra-pr:14581 server local |
69001be to
9840c9e
Compare
9840c9e to
d76ca3f
Compare
There was a problem hiding this comment.
I've swiftly glanced through the code - that part looks alright so far in terms you're on point when it comes to things that need changing. Would have to give it a more thorough look once it's ready for review, especially because there's a lot of files affected by the PR.
Speaking about the behavior - I do like it very much, makes things easier for users to get the idea of what Kestra is and how it's supposed to work.
Below are some minor details I've notice during the test:
- This badge is not visible enough on dark mode
-
Starting the tutorial shows the dialog before the redirection to editor
Screencast from 2026-02-13 09-14-04.webm -
There is a
Completelabel at the bottom of dialog which I'm not sure what's the purpose of
Okay, I figured out after the step where I had to type something in, maybe a bit of design change where we don't show it when there's no user interaction needed.
- It was not clear to me that I should type the
idfield because the editor is not in the foreground here, but looks like it's either disabled or only in the background
- There's console errors when I moved to this step
- Here, a little highlighting of
Savebutton might not hurt
-
Same goes for the next step about executing.
-
Overlapping of dialogs - not good
Overall, beside those little things, this was an excellent job so far @proddata! 🚀
|
And this PR will also close #9874, so I'll add it into description. |
6e15404 to
d7e15e8
Compare
…boarding i18n keys remove legacy VueTour flow and obsolete onboarding button components wire onboarding v2 guided/self-serve flow in create/edit journey migrate first-flow copy to onboarding.* and clean unused en translation keys update onboarding and flow-save unit tests
d7e15e8 to
ddc23f8
Compare
resolves #14558
Closes #9874.