Skip to content

Initial Perf Analysis Diff UI - #189

Open
markerikson wants to merge 32 commits into
mainfrom
feature/PRO-913-perf-analysis-ui
Open

markerikson wants to merge 32 commits into
mainfrom
feature/PRO-913-perf-analysis-ui

Conversation

@markerikson

@markerikson markerikson commented Dec 13, 2024 •

Copy link
Copy Markdown
Collaborator

This PR:

  • Sets up Shadcn UI dependencies (since v0 is fond of using Shadcn for any UI it generates)
  • Updates the perf analysis types copied from the backend
  • Disables TS array index checks (which were turned on for some reason)
  • Adds logic from Andarist to diff perf results
  • Builds out an initial perf analysis UI:
    • Fetches the initial perf result for that recording ID
    • Fetches the corresponding workspace tracking file, and finds other recent results from main
    • Currently passes just two results to be diffed on the client side
    • Displays some initial diff results, starting from a v0-generated layout

Example: https://dashboard-git-feature-pro-913-perf-analysis-ui-replayio.vercel.app/performance-analysis-diff?0ec0c62a-5a9f-41d9-9b2b-5304198ddda1

image

@vercel

vercel Bot commented Dec 13, 2024 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
dashboard ✅ Ready (Inspect) Visit Preview Dec 18, 2024 9:33pm

@replay-io

replay-io Bot commented Dec 13, 2024 •

Copy link
Copy Markdown

E2E Tests

Status Complete ↗︎
Commit 46d0224
Results
✅ 36 Passed
  • authentication: should require authentication to view the library
  • create-standard-team-fails: shows mutation error message
  • create-standard-team: create a standard workspace for debugging
  • create-test-suites-team-fails: shows mutation error message
  • create-test-suites-team: create a test suites workspace
  • login-external-browser: should login using an external browser
  • login-logout-devtools: should login and logout from devtools
  • login-logout-library: should login and logout from the library
  • my-library: filtering and share dialog
  • releases: should list all releases, with the most recent ones at the top
  • remember-last-path: requests to the root path should auto re-direct back to the most recently viewed page
  • team-404-page: should be shown when a user does not have access to a team
  • team-library: filtering and share dialog
  • team-settings-billing-subscription-active-no-card: should show option to add payment method when missing (Replay 1, Replay 2)
  • team-settings-billing-subscription-active: should show information about the subscription and payment method on file (Replay 1, Replay 2)
  • team-settings-billing-trial: should prompt the user to add payment information if none has been entered (Replay 1, Replay 2)
  • team-settings-default-tab-subscription-canceled-no-payment: should show the billing tab by default for canceled subscriptions (Replay 1, Replay 2)
  • team-settings-default-tab-subscription-canceled: should show the billing tab by default for canceled subscriptions (Replay 1, Replay 2)
  • team-settings-default-tab-subscription-canceled: should show the billing tab by default for canceled subscriptions
  • team-settings-members-pending-access-request: should show users who have requested access to the team
  • team-settings-members: should label member roles
  • team-settings-protected-routes: should restrict certain routes from non-developer or non-admin roles
  • test-suites-runs-1: text and drop-down filters
  • test-suites-runs-2: passed run in main branch with source
  • test-suites-runs-3: failed run in temp branch without source
  • test-suites-runs-4: should respect workspace retention limits
  • test-suites-runs-5: should handle when filters hide a selected test run or test
  • test-suites-runs-6: should show a warning when a user opens a URL linking to a test run that could not be loaded
  • test-suites-runs-6: should show a warning when a user opens a URL linking to a test that could not be loaded
  • test-suites-runs-8: should explain missing recording if within workspace retention limits
  • test-suites-tests-1: sorting and filtering test runs
  • test-suites-tests-2: failed test executions
  • test-suites-tests-3: flaky test executions
  • test-suites-tests-4: should respect workspace retention limits
  • test-suites-tests-5: should handle when filters hide a selected test
  • test-suites-tests-6: should show a warning when a user opens a URL linking to a test that could not be loaded
  • @markerikson
    markerikson force-pushed the feature/PRO-913-perf-analysis-ui branch from e3a87e5 to 9849830 Compare December 18, 2024 19:51
    @markerikson markerikson changed the title [DRAFT] Perf Analysis Diff UI Initial Perf Analysis Diff UI Dec 18, 2024
    @markerikson
    markerikson marked this pull request as ready for review December 18, 2024 21:11

    This branch was successfully deployed

    1 active deployment
    Preview — 46d02244 Deployed Dec 18, 2024 by vercel[bot]
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Labels

    None yet

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    1 participant