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(insights): add transaction action buttons to transaction summary #80735

Merged
merged 2 commits into from
Nov 14, 2024

Conversation

DominikB2014
Copy link
Contributor

@DominikB2014 DominikB2014 commented Nov 14, 2024

This PR adds the transaction action buttons to the top right of the domain view transaction summary

(Create alert, starred for team, etc)
image

@DominikB2014 DominikB2014 requested a review from a team as a code owner November 14, 2024 15:18
@DominikB2014 DominikB2014 requested review from gggritso and a team November 14, 2024 15:18
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 14, 2024
Copy link

codecov bot commented Nov 14, 2024

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
8224 1 8223 0
View the top 1 failed tests by shortest run time
WidgetBuilder discover dataset split events-stats selects the transaction discover split type as the dataset when the request completes
Stack Traces | 0.947s run time
Error: expect(element).toBeChecked()

Received element is not checked:
  <input aria-label="Transactions" class="css-141lam5-Radio-checkedCss e72tocb0" type="radio" />
    at Object.<anonymous> (.../dashboards/widgetBuilder/widgetBuilder.spec.tsx:1820:13)
    at runNextTicks (node:internal/process/task_queues:60:5)
    at processTimers (node:internal/timers:511:9)

To view more test analytics, go to the Test Analytics Dashboard
Got feedback? Let us know on Github

@DominikB2014 DominikB2014 merged commit 542646a into master Nov 14, 2024
43 of 44 checks passed
@DominikB2014 DominikB2014 deleted the DominikB2014/add-back-transaction-buttons branch November 14, 2024 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants