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.

Sign up for GitHub

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

Jump to bottom

🖼 459 adding new test details and trace view pages #474

Merged
merged 2 commits into from
May 11, 2022
Merged

🖼 459 adding new test details and trace view pages #474

merged 2 commits into from
May 11, 2022

Conversation

xoscar
Copy link
Collaborator

@xoscar xoscar commented May 11, 2022

This PR adds the new styles for the test details and trace view.

Changes

  • New route only for trace views.
  • Adding trace header for both tests and traces.
  • Updating the timeline chart
  • Updating the DAG.
  • Create test flows now redirect directly to the test result page.
  • New storybook components.

Fixes

Checklist

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

https://www.loom.com/share/8e875414cb6049d3b08eee700f0a74e9

@xoscar xoscar added this to the MVProduct milestone May 11, 2022
@xoscar xoscar self-assigned this May 11, 2022
@xoscar xoscar linked an issue May 11, 2022 that may be closed by this pull request
[New UX] Trace View #1 (Base page, Diagrams) #459
Closed
kdhamric
kdhamric approved these changes May 11, 2022
View reviewed changes
Copy link
Collaborator

@kdhamric kdhamric left a comment

Choose a reason for hiding this comment

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

Watched the Loom video (thanks!) - looking great

@xoscar xoscar merged commit b89e883 into main May 11, 2022
@xoscar xoscar deleted the 459-new-ux-trace-view-1-base-page-diagrams branch May 11, 2022 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kdhamric kdhamric kdhamric approved these changes

@cescoferraro cescoferraro Awaiting requested review from cescoferraro

Assignees

@xoscar xoscar

Labels
frontend
Projects
None yet
Milestone
MVProduct
Development

Successfully merging this pull request may close these issues.

[New UX] Trace View #1 (Base page, Diagrams)
2 participants
@xoscar @kdhamric