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(frontend): add wizard analytics #3543

Merged
merged 3 commits into from
Jan 18, 2024
Merged

Conversation

jorgeepc
Copy link
Contributor

This PR adds the wizard analytics and the success criteria logic for the Tracing Backend step.

Changes

  • wizard analytics
  • success criteria logic for the Tracing Backend step

Fixes

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

@jorgeepc jorgeepc self-assigned this Jan 18, 2024
@jorgeepc jorgeepc marked this pull request as ready for review January 18, 2024 21:18
testOtlpConnection,
{isLoading: isOtlpTestConnectionLoading, data: otlpTestConnectionResponse, isError: isOtlpTestConnectionError},
] = useLazyTestOtlpConnectionQuery({
pollingInterval,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Good approach!

@jorgeepc jorgeepc merged commit 27594c1 into feat-wizard Jan 18, 2024
37 checks passed
@jorgeepc jorgeepc deleted the feat-wizard-analytics branch January 18, 2024 21:54
xoscar added a commit that referenced this pull request Jan 23, 2024
* feat(frontend): add wizard layout and architecture (#3521)

* feat(frontend): add wizard layout and architecture

* add home icon

* fix cypress tests

* Feat wizard structure (#3523)

feat(frontend): wizard structure

* Feat wizard tracing backend (#3525)

* feat(frontend): Wizard - Tracing Backend UI

* add customization wrapper

---------

Co-authored-by: Jorge Padilla <jorge.esteban.padilla@gmail.com>

* feat(frontend): add wizard ui improvements (#3530)

* feat(frontend): add wizard ui improvements

* fix import

* feat: wizard backend (#3531)

* fix(docs): E2E docs fix

* feat: Wizard wire up

* feat: Wizard wire up

* fixing unit tests

* pr suggestions

* feat(frontend): Wizard - Create Test Flow (#3532)

* fix(docs): E2E docs fix

* feat: Wizard wire up

* feat: Wizard wire up

* fixing unit tests

* feat(frontend): Wizard - Create Test Flow

* feat(frontend): Wizard - Create Test Flow

* feat(frontend): Wizard - Create Test Flow

* fixing build

* fix(backend): wizard crud (#3535)

fix(backend): fixing wizard crud

* feat(frontend): update wizard steps state (#3537)

* feat(frontend): Wizard - OTLP Test connection (#3540)

* feat(frontend): add wizard analytics (#3543)

* feat(frontend): Wizard - Completion Steps Wire-Up (#3546)

* feat(frontend): Wizard - Completion Steps Wire-Up

* fix wrapper level

* adding analytics event

* fixing wrong loading state

* chore: Updating default wizard steps (#3548)

Co-authored-by: Oscar <oscar@MacBook-Pro-de-Oscar.local>

* fix(frontend): fix next step for tracing backend (#3547)

* fix(frontend): fix selected tracing backend (#3551)

---------

Co-authored-by: Jorge Padilla <jorge.esteban.padilla@gmail.com>
Co-authored-by: Oscar <oscar@MacBook-Pro-de-Oscar.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants