Skip to content

ref(integrations): Replace useApi with useMutation for external mapping deletes#112908

Merged
priscilawebdev merged 1 commit intomasterfrom
priscila/ref/migrate-external-mappings-useapi
Apr 14, 2026
Merged

ref(integrations): Replace useApi with useMutation for external mapping deletes#112908
priscilawebdev merged 1 commit intomasterfrom
priscila/ref/migrate-external-mappings-useapi

Conversation

@priscilawebdev
Copy link
Copy Markdown
Member

Migrate DELETE operations in IntegrationExternalUserMappings and
IntegrationExternalTeamMappings from legacy useApi/requestPromise
to useMutation/fetchMutation from TanStack Query.

This is part of the ongoing effort to remove legacy useApi usage
in favor of TanStack Query mutations. The read-side queries (useApiQuery)
and the form component (integrationExternalMappingForm.tsx) were already
migrated — these two wrapper files were the remaining holdouts.

…ng deletes

Migrate DELETE operations in IntegrationExternalUserMappings and
IntegrationExternalTeamMappings from legacy useApi/requestPromise
to useMutation/fetchMutation from TanStack Query.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 14, 2026
@sentry
Copy link
Copy Markdown
Contributor

sentry bot commented Apr 14, 2026

Sentry Snapshot Testing

Name Added Removed Modified Renamed Unchanged Status
sentry-frontend
sentry-frontend
0 0 0 0 204 ✅ Unchanged

@priscilawebdev
Copy link
Copy Markdown
Member Author

bugbot run

Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit f3c4377. Configure here.

@priscilawebdev priscilawebdev marked this pull request as ready for review April 14, 2026 12:03
@priscilawebdev priscilawebdev requested a review from a team as a code owner April 14, 2026 12:03
Copy link
Copy Markdown
Collaborator

@TkDodo TkDodo left a comment

Choose a reason for hiding this comment

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

🔥

@priscilawebdev priscilawebdev merged commit 2ec6f35 into master Apr 14, 2026
66 checks passed
@priscilawebdev priscilawebdev deleted the priscila/ref/migrate-external-mappings-useapi branch April 14, 2026 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants