Skip to content

ref(notebooks): Let the update validator archive or update a notebook - #130824

Merged
wedamija merged 1 commit into
masterfrom
danf/notebooks-details-put-validator
Nov 14, 2026
Merged

wedamija merged 1 commit into
masterfrom
danf/notebooks-details-put-validator

Conversation

@wedamija

Copy link
Copy Markdown
Member

We're moving the notebook tables (Investigation*) into a new sentry.notebooks app, separate from the hypothesis engine that writes them, so other Seer producers like Autofix and triage can write notebooks too.

NotebookUpdateValidator.save() picks archive or update from the validated fields, so the details endpoint PUT doesn't branch on the payload. Responses don't change.

@wedamija
wedamija requested review from a team as code owners November 11, 2026 19:40
@wedamija
wedamija requested review from arslnb and billyvg November 11, 2026 19:40
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 11, 2026
@wedamija
wedamija force-pushed the danf/notebooks-public branch from 9ca5197 to e573180 Compare November 13, 2026 18:37
@wedamija
wedamija requested review from a team as code owners November 13, 2026 18:37
@wedamija
wedamija force-pushed the danf/notebooks-details-put-validator branch from 238dace to defcdb4 Compare November 13, 2026 18:37
@sentry

sentry Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Sentry Snapshot Testing

Name Added Removed Changed Renamed Unchanged Skipped Status
sentry-frontend
sentry-frontend
0 0 0 0 934 0 ✅ Unchanged

⚙️ sentry-frontend Snapshot Settings

@wedamija
wedamija requested a review from billyvg November 13, 2026 18:44
@wedamija
wedamija force-pushed the danf/notebooks-public branch from e573180 to 91ebb4b Compare November 13, 2026 19:15
@wedamija
wedamija force-pushed the danf/notebooks-details-put-validator branch from defcdb4 to 65db876 Compare November 13, 2026 19:15
@wedamija
wedamija removed the request for review from billyvg November 13, 2026 22:10
@wedamija
wedamija force-pushed the danf/notebooks-public branch from 91ebb4b to 8fcfa93 Compare November 13, 2026 22:37
@wedamija
wedamija force-pushed the danf/notebooks-details-put-validator branch from 65db876 to 51eb023 Compare November 13, 2026 22:37
@wedamija
wedamija force-pushed the danf/notebooks-public branch from 8fcfa93 to d3f754e Compare November 13, 2026 23:30
@wedamija
wedamija force-pushed the danf/notebooks-details-put-validator branch from 51eb023 to 57e346d Compare November 13, 2026 23:30
@wedamija
wedamija force-pushed the danf/notebooks-public branch from d3f754e to 5227fe3 Compare November 13, 2026 23:43
Base automatically changed from danf/notebooks-public to master November 14, 2026 01:11
NotebookUpdateValidator.save() picks archive or update from the validated
fields, so the details endpoint PUT doesn't branch on the payload.
@wedamija
wedamija force-pushed the danf/notebooks-details-put-validator branch from 57e346d to 21b4b68 Compare November 14, 2026 01:12
@wedamija
wedamija merged commit 73b5540 into master Nov 14, 2026
79 checks passed
@wedamija
wedamija deleted the danf/notebooks-details-put-validator branch November 14, 2026 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants