Commit ecd19d5
authored
feat(snapshots): Show diff_threshold in snapshot detail header (#112740)
<img width="1426" height="300" alt="CleanShot 2026-04-10 at 14 08 50@2x"
src="https://github.com/user-attachments/assets/89acfcf9-9305-4f5b-861a-ae665d01acf9"
/>
## Summary
Surfaces the optional `diff_threshold` value from the snapshot manifest
in the snapshot detail API response and renders it in the header UI next
to the branch name when set.
- **Backend**: Added `diff_threshold` field to
`SnapshotDetailsApiResponse` model, populated from the manifest
- **Frontend**: Added `diff_threshold` to the TypeScript type, rendered
with a settings icon as "Diff threshold: X.X%" in the header
When `diff_threshold` is not set (most snapshots), nothing changes in
the UI.
Refs EME-406
## Test plan
- [ ] Load a snapshot that has `diff_threshold` set in its manifest —
verify it shows "Diff threshold: X.X%" next to the branch name
- [ ] Load a snapshot without `diff_threshold` — verify no extra element
appears in the header1 parent c0d6db4 commit ecd19d5
File tree
4 files changed
+25
-2
lines changed- src/sentry/preprod/api
- endpoints
- models/snapshots
- static/app/views/preprod
- snapshots/header
- types
4 files changed
+25
-2
lines changedLines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
435 | 435 | | |
436 | 436 | | |
437 | 437 | | |
| 438 | + | |
438 | 439 | | |
439 | 440 | | |
440 | 441 | | |
| |||
Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
97 | 97 | | |
98 | 98 | | |
99 | 99 | | |
| 100 | + | |
| 101 | + | |
100 | 102 | | |
101 | 103 | | |
Lines changed: 20 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
11 | 17 | | |
12 | 18 | | |
13 | 19 | | |
| |||
66 | 72 | | |
67 | 73 | | |
68 | 74 | | |
69 | | - | |
| 75 | + | |
| 76 | + | |
70 | 77 | | |
71 | 78 | | |
72 | 79 | | |
| |||
82 | 89 | | |
83 | 90 | | |
84 | 91 | | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
85 | 103 | | |
86 | 104 | | |
87 | 105 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
| 55 | + | |
| 56 | + | |
55 | 57 | | |
56 | 58 | | |
57 | 59 | | |
| |||
0 commit comments