ref: replace withOrganization with useOrganization in function components#115343
Merged
evanpurkhiser merged 1 commit intoMay 21, 2026
Conversation
github-actions
Bot
added
the
Scope: Frontend
Contributor
📊 Type Coverage Diff✅ No new type safety issues introduced. Coverage: 93.61% |
evanpurkhiser
force-pushed
the
evanpurkhiser/ref-replace-withorganization-with-useorganization-in-function-components
branch
from
d012250 to
5caa581
Compare
github-actions
Bot
added
the
Scope: Backend
Contributor
|
🚨 Warning: This pull request contains Frontend and Backend changes! It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently. Have questions? Please ask in the |
Contributor
Backend Test FailuresFailures on
|
evanpurkhiser
force-pushed
the
evanpurkhiser/ref-replace-withorganization-with-useorganization-in-function-components
branch
from
5caa581 to
6073dc0
Compare
…ents Swap the 17 trivial (already-FC) consumers of withOrganization to call useOrganization directly. The HoC itself stays — the class-component consumers will need conversion first. Where withOrganization was the only HoC wrapping the export, the file moves to a named export (and its importers update to match) because the no-default-exports eslint rule disallows plain-identifier default exports.
evanpurkhiser
force-pushed
the
evanpurkhiser/ref-replace-withorganization-with-useorganization-in-function-components
branch
from
6073dc0 to
54da118
Compare
roggenkemper
approved these changes
May 21, 2026
evanpurkhiser
deleted the
evanpurkhiser/ref-replace-withorganization-with-useorganization-in-function-components
branch
JonasBa
pushed a commit
that referenced
this pull request
May 21, 2026
…ents (#115343) Swap the 17 trivial (already-FC) consumers of `withOrganization` to call `useOrganization` directly. The HoC itself stays — the class-component consumers will need conversion first. Where `withOrganization` was the only HoC wrapping the export, the file moves to a named export (and its importers update to match) because the `no-default-exports` eslint rule disallows plain-identifier default exports. Two of those (`organizationAuthTokens/index.tsx` and `organizationIntegrations/pluginDetailedView.tsx`) are lazy-imported in `router/routes.tsx`, so the lazy import unwraps the named export into a `{default: ...}` shape. Follow-up to #115334 (drop withSentryAppComponents HoC), continuing the legacy HoC cleanup.
natemoo-re
pushed a commit
that referenced
this pull request
May 21, 2026
…ents (#115343) Swap the 17 trivial (already-FC) consumers of `withOrganization` to call `useOrganization` directly. The HoC itself stays — the class-component consumers will need conversion first. Where `withOrganization` was the only HoC wrapping the export, the file moves to a named export (and its importers update to match) because the `no-default-exports` eslint rule disallows plain-identifier default exports. Two of those (`organizationAuthTokens/index.tsx` and `organizationIntegrations/pluginDetailedView.tsx`) are lazy-imported in `router/routes.tsx`, so the lazy import unwraps the named export into a `{default: ...}` shape. Follow-up to #115334 (drop withSentryAppComponents HoC), continuing the legacy HoC cleanup.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Swap the 17 trivial (already-FC) consumers of
withOrganizationto calluseOrganizationdirectly. The HoC itself stays — the class-component consumers will need conversion first.Where
withOrganizationwas the only HoC wrapping the export, the file moves to a named export (and its importers update to match) because theno-default-exportseslint rule disallows plain-identifier default exports. Two of those (organizationAuthTokens/index.tsxandorganizationIntegrations/pluginDetailedView.tsx) are lazy-imported inrouter/routes.tsx, so the lazy import unwraps the named export into a{default: ...}shape.Follow-up to #115334 (drop withSentryAppComponents HoC), continuing the legacy HoC cleanup.