Skip to content
This repository was archived by the owner on Aug 14, 2024. It is now read-only.

Add UI event transactions#766

Merged
philipphofmann merged 20 commits intomasterfrom
feat/ui-event-transactions
Nov 21, 2022
Merged

Add UI event transactions#766
philipphofmann merged 20 commits intomasterfrom
feat/ui-event-transactions

Conversation

@philipphofmann
Copy link
Copy Markdown
Member

@philipphofmann philipphofmann commented Nov 17, 2022

Add specification for UI event transactions with Gherkin syntax.

Fixes GH-647

Add specification for UI event transactions with Gherkin syntax.
@vercel
Copy link
Copy Markdown

vercel bot commented Nov 17, 2022

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

Name Status Preview Updated
develop ✅ Ready (Inspect) Visit Preview Nov 21, 2022 at 7:57AM (UTC)

@philipphofmann philipphofmann changed the title Add docs for UI event transactions Add UI event transactions Nov 17, 2022
Copy link
Copy Markdown
Member

@romtsn romtsn left a comment

Choose a reason for hiding this comment

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

We also have a boolean option in SentryOptions (enableUserInteractionTracing), should we mentioned it here?

Co-authored-by: Roman Zavarnitsyn <rom4ek93@gmail.com>
Copy link
Copy Markdown
Contributor

@marandaneto marandaneto left a comment

Choose a reason for hiding this comment

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

LGTM @philipphofmann and once more, thank you for doing this.

Co-authored-by: Roman Zavarnitsyn <rom4ek93@gmail.com>
philipphofmann and others added 4 commits November 18, 2022 14:45
Co-authored-by: Roman Zavarnitsyn <rom4ek93@gmail.com>
Co-authored-by: Roman Zavarnitsyn <rom4ek93@gmail.com>
Co-authored-by: Roman Zavarnitsyn <rom4ek93@gmail.com>
Copy link
Copy Markdown
Member

@romtsn romtsn left a comment

Choose a reason for hiding this comment

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

2 minor fixes + still missing enableUserInteractionTracing option. Otherwise, LGTM 👍 thanks for doing that

philipphofmann and others added 2 commits November 21, 2022 08:47
Co-authored-by: Roman Zavarnitsyn <rom4ek93@gmail.com>
Co-authored-by: Roman Zavarnitsyn <rom4ek93@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document UI Event Transaction

3 participants