Unreal: Add manual plugin config example and highlight `SendDefaultPii` usage #89849

tustanivsky · 2025-04-17T12:34:18Z

This PR adds an example of manual plugin configuration for the Unreal Engine project wizard and highlights the use of the SendDefaultPii option (getsentry/team-sdks#121).

Related UE plugin documentation update: getsentry/sentry-docs#13435

This also highlights `SendDefaultPii` setting usage

static/app/gettingStartedDocs/unreal/unreal.tsx

Co-authored-by: Simon Hellmayr <shellmayr@users.noreply.github.com>

static/app/gettingStartedDocs/unreal/unreal.tsx

tustanivsky · 2025-04-22T12:31:07Z

Lint errors were fixed in 81a3c39

static/app/gettingStartedDocs/unreal/unreal.tsx

Co-authored-by: Priscila Oliveira <priscilawebdev@gmail.com>

tustanivsky · 2025-04-24T10:12:05Z

@priscilawebdev Can certain CI checks (like Vercel deployment or dispatch) fail because this PR comes from a public fork? It seems these require approval from someone at getsentry with write access to this repo.

priscilawebdev · 2025-04-24T10:48:31Z

@priscilawebdev Can certain CI checks (like Vercel deployment or dispatch) fail because this PR comes from a public fork? It seems these require approval from someone at getsentry with write access to this repo.

yes! I just triggered the tests for this PR 😉

priscilawebdev · 2025-04-24T10:49:26Z

I approved Vercel deployment but it does not seem to work 🤔

bruno-garcia · 2025-04-28T18:35:44Z

static/app/gettingStartedDocs/unreal/unreal.tsx

+{
+    Settings->Dsn = TEXT("${params.dsn.public}");
+
+    // Add data like request headers, user ip address, device name, etc.


There's no request header in Unreal though. What would be the right comment?

Suggested change

// Add data like request headers, user ip address, device name, etc.

// Add data like ip address and device name

We should link to the docs page we're creating where we list what this is

Good point, I've just added "Data Collected" page to docs (https://docs.sentry.io/platforms/unreal/data-management/data-collected/) and will link it here as well

bruno-garcia · 2025-04-28T18:36:13Z

Approved vercel preview, lets see if that works

bruno-garcia · 2025-04-28T21:08:02Z

I approved Vercel deployment but it does not seem to work 🤔

~~yeah me too~~

Might be now because there was another commit

bruno-garcia · 2025-04-28T21:11:27Z

Vercel running again

bruno-garcia · 2025-04-29T15:33:37Z

Run python3 -uS .github/workflows/scripts/getsentry-dispatch-setup
permissions failed!

has write permission: False
has [Trigger: getsentry tests] label: False
args.label_names: ['Scope: Frontend']
please have a collaborator add the [Trigger: getsentry tests] label once they have reviewed the code to trigger downstream tests.
Error: Process completed with exit code 1.

…i` usage (#89849) This PR adds an example of manual plugin configuration for the Unreal Engine project wizard and highlights the use of the `SendDefaultPii` option (getsentry/team-sdks#121). Related UE plugin documentation update: getsentry/sentry-docs#13435 --------- Co-authored-by: Simon Hellmayr <shellmayr@users.noreply.github.com> Co-authored-by: Priscila Oliveira <priscilawebdev@gmail.com>

Add manual configuration step to Unreal Engine wizard

4a98096

This also highlights `SendDefaultPii` setting usage

tustanivsky requested a review from a team as a code owner April 17, 2025 12:34

github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 17, 2025

Flash0ver reviewed Apr 17, 2025

View reviewed changes

static/app/gettingStartedDocs/unreal/unreal.tsx Outdated Show resolved Hide resolved

Flash0ver reviewed Apr 17, 2025

View reviewed changes

static/app/gettingStartedDocs/unreal/unreal.tsx Outdated Show resolved Hide resolved

tustanivsky added 2 commits April 17, 2025 16:27

Update unreal.tsx

97c32f8

Merge branch 'master' into unreal/send-default-pii

3debb6a

shellmayr reviewed Apr 22, 2025

View reviewed changes

static/app/gettingStartedDocs/unreal/unreal.tsx Outdated Show resolved Hide resolved

Update static/app/gettingStartedDocs/unreal/unreal.tsx

a042706

Co-authored-by: Simon Hellmayr <shellmayr@users.noreply.github.com>

shellmayr reviewed Apr 22, 2025

View reviewed changes

static/app/gettingStartedDocs/unreal/unreal.tsx Outdated Show resolved Hide resolved

tustanivsky added 4 commits April 22, 2025 09:53

Add DSN param to configure settings sniuppet

46c1568

Try using fragment

f983bc9

Fix lint

81a3c39

Merge branch 'master' into unreal/send-default-pii

63bc84a

Merge branch 'master' into unreal/send-default-pii

842acbe

priscilawebdev reviewed Apr 24, 2025

View reviewed changes

static/app/gettingStartedDocs/unreal/unreal.tsx Outdated Show resolved Hide resolved

static/app/gettingStartedDocs/unreal/unreal.tsx Outdated Show resolved Hide resolved

tustanivsky and others added 4 commits April 24, 2025 13:02

Update static/app/gettingStartedDocs/unreal/unreal.tsx

e0f35eb

Co-authored-by: Priscila Oliveira <priscilawebdev@gmail.com>

Update unreal.tsx

d757651

Merge branch 'master' into unreal/send-default-pii

1f78d9f

Fix string

62352e9

priscilawebdev added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Apr 24, 2025

Merge branch 'master' into unreal/send-default-pii

53ff886

github-actions bot removed the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Apr 28, 2025

bruno-garcia reviewed Apr 28, 2025

View reviewed changes

vercel bot deployed to Preview April 28, 2025 18:40 View deployment

Update unreal.tsx

8774af3

bruno-garcia approved these changes Apr 28, 2025

View reviewed changes

vercel bot deployed to Preview April 28, 2025 21:13 View deployment

tustanivsky added 3 commits April 29, 2025 05:53

Merge branch 'master' into unreal/send-default-pii

ca70be0

Merge branch 'master' into unreal/send-default-pii

9460edd

Merge branch 'master' into unreal/send-default-pii

dcb1f2f

bruno-garcia enabled auto-merge (squash) April 29, 2025 15:19

Merge branch 'master' into unreal/send-default-pii

7a4f608

vercel bot deployed to Preview April 29, 2025 15:24 View deployment

colin-sentry added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Apr 29, 2025

bruno-garcia merged commit 7ec413a into getsentry:master Apr 29, 2025
45 of 46 checks passed

github-actions bot locked and limited conversation to collaborators May 15, 2025

tustanivsky deleted the unreal/send-default-pii branch September 8, 2025 05:57

	// Add data like request headers, user ip address, device name, etc.
	// Add data like ip address and device name

Uh oh!

Unreal: Add manual plugin config example and highlight SendDefaultPii usage #89849

Unreal: Add manual plugin config example and highlight SendDefaultPii usage #89849

Uh oh!

Conversation

tustanivsky commented Apr 17, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tustanivsky commented Apr 22, 2025

Uh oh!

Uh oh!

Uh oh!

tustanivsky commented Apr 24, 2025

Uh oh!

priscilawebdev commented Apr 24, 2025

Uh oh!

priscilawebdev commented Apr 24, 2025

Uh oh!

bruno-garcia Apr 28, 2025

Choose a reason for hiding this comment

Uh oh!

tustanivsky Apr 28, 2025

Choose a reason for hiding this comment

Uh oh!

bruno-garcia commented Apr 28, 2025

Uh oh!

bruno-garcia commented Apr 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

bruno-garcia commented Apr 28, 2025

Uh oh!

bruno-garcia commented Apr 29, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Unreal: Add manual plugin config example and highlight `SendDefaultPii` usage #89849

Unreal: Add manual plugin config example and highlight `SendDefaultPii` usage #89849

bruno-garcia commented Apr 28, 2025 •

edited

Loading