Add hasSchemaFormConfig to Sentry client payload and update related… - #571
Conversation
… test assertions
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR updates the Sentry client’s issue alert rule payload to explicitly send a hasSchemaFormConfig flag and aligns the feature test expectations with this new contract. Sequence diagram for Sentry issue alert rule creation including hasSchemaFormConfigsequenceDiagram
participant App as App
participant SentryClient as SentryClient
participant SentryAPI as SentryAPI
App->>SentryClient: createIssueAlertRule(projectSlug, priorityId)
SentryClient->>SentryAPI: POST /projects/{projectSlug}/alerts/rules
activate SentryAPI
Note over SentryClient,SentryAPI: Payload.customSettings includes hasSchemaFormConfig = true
SentryAPI-->>SentryClient: 201 Created
SentryClient-->>App: Issue alert rule created response
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| PHP | Jun 2, 2026 5:38p.m. | Review ↗ | |
| JavaScript | Jun 2, 2026 5:38p.m. | Review ↗ | |
| Python | Jun 2, 2026 5:38p.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
There was a problem hiding this comment.
Pull request overview
This PR updates the Sentry issue alert rule creation payload to include a hasSchemaFormConfig flag, and adjusts the outbound sync feature test to assert that the flag is present and set to true.
Changes:
- Add
hasSchemaFormConfig: trueto the Sentry action payload when creating issue alert rules. - Update the outbound sync feature test to assert
hasSchemaFormConfigis included andtrue.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
app/Integrations/Sentry/Services/SentryClient.php |
Adds hasSchemaFormConfig to the Sentry issue alert rule action payload. |
tests/Feature/Integrations/Sentry/OutboundSyncTest.php |
Updates assertions to expect hasSchemaFormConfig === true in the outbound request payload. |
… test assertions
Summary by Sourcery
Include the hasSchemaFormConfig flag in Sentry issue alert rule payloads and update tests to assert its presence.
New Features:
Tests: