Skip to content

Commit d8b067c

Browse files
committed
Add comments to clarify context usage
1 parent eccdbdf commit d8b067c

File tree

6 files changed

+13
-6
lines changed

6 files changed

+13
-6
lines changed

packages/clerk-js/src/ui/components/PaymentAttempts/PaymentAttemptPage.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,11 @@ import { useRouter } from '../../router';
2929
export const PaymentAttemptPage = () => {
3030
const { params, navigate } = useRouter();
3131
const subscriberType = useSubscriberTypeContext();
32-
const organizationCtx = useOrganizationContext();
3332
const localizationRoot = useSubscriberTypeLocalizationRoot();
3433
const { t, translateError } = useLocalizations();
3534
const clerk = useClerk();
35+
// Do not use `useOrganization` to avoid triggering the in-app enable organizations prompt in development instance
36+
const organizationCtx = useOrganizationContext();
3637

3738
const {
3839
data: paymentAttempt,

packages/clerk-js/src/ui/components/PaymentMethods/PaymentMethods.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,11 +60,12 @@ const RemoveScreen = ({
6060
const { close } = useActionContext();
6161
const card = useCardState();
6262
const subscriberType = useSubscriberTypeContext();
63-
const organizationCtx = useOrganizationContext();
6463
const localizationRoot = useSubscriberTypeLocalizationRoot();
6564
const ref = useRef(
6665
`${paymentMethod.paymentType === 'card' ? paymentMethod.cardType : paymentMethod.paymentType} ${paymentMethod.paymentType === 'card' ? `⋯ ${paymentMethod.last4}` : '-'}`,
6766
);
67+
// Do not use `useOrganization` to avoid triggering the in-app enable organizations prompt in development instance
68+
const organizationCtx = useOrganizationContext();
6869

6970
if (!ref.current) {
7071
return null;
@@ -196,9 +197,10 @@ const PaymentMethodMenu = ({
196197
}) => {
197198
const { open } = useActionContext();
198199
const card = useCardState();
199-
const organizationCtx = useOrganizationContext();
200200
const subscriberType = useSubscriberTypeContext();
201201
const localizationRoot = useSubscriberTypeLocalizationRoot();
202+
// Do not use `useOrganization` to avoid triggering the in-app enable organizations prompt in development instance
203+
const organizationCtx = useOrganizationContext();
202204

203205
const actions = [
204206
{

packages/clerk-js/src/ui/components/PricingTable/PricingTableDefault.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,7 @@ function Card(props: CardProps) {
104104
const { isSignedIn } = useSession();
105105
const { mode = 'mounted', ctaPosition: ctxCtaPosition } = usePricingTableContext();
106106
const subscriberType = useSubscriberTypeContext();
107+
// Do not use `useOrganization` to avoid triggering the in-app enable organizations prompt in development instance
107108
const organizationCtx = useOrganizationContext();
108109

109110
const ctaPosition = pricingTableProps.ctaPosition || ctxCtaPosition || 'bottom';

packages/clerk-js/src/ui/components/Statements/StatementPage.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,11 @@ import { Statement } from './Statement';
2323
export const StatementPage = () => {
2424
const { params, navigate } = useRouter();
2525
const subscriberType = useSubscriberTypeContext();
26-
const organizationCtx = useOrganizationContext();
2726
const localizationRoot = useSubscriberTypeLocalizationRoot();
2827
const { t, translateError } = useLocalizations();
2928
const clerk = useClerk();
29+
// Do not use `useOrganization` to avoid triggering the in-app enable organizations prompt in development instance
30+
const organizationCtx = useOrganizationContext();
3031

3132
const {
3233
data: statement,

packages/clerk-js/src/ui/components/SubscriptionDetails/index.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,6 @@ const SubscriptionDetailsInternal = (props: __internal_SubscriptionDetailsProps)
173173

174174
const SubscriptionDetailsFooter = withCardStateProvider(() => {
175175
const subscriberType = useSubscriberTypeContext();
176-
const organizationCtx = useOrganizationContext();
177176
const { isLoading, error, setError, setLoading, setIdle } = useCardState();
178177
const {
179178
subscription: selectedSubscription,
@@ -183,6 +182,8 @@ const SubscriptionDetailsFooter = withCardStateProvider(() => {
183182
const { data: subscription } = useSubscription();
184183
const { setIsOpen } = useDrawerContext();
185184
const { onSubscriptionCancel } = useSubscriptionDetailsContext();
185+
// Do not use `useOrganization` to avoid triggering the in-app enable organizations prompt in development instance
186+
const organizationCtx = useOrganizationContext();
186187

187188
const onOpenChange = useCallback((open: boolean) => setConfirmationOpen(open), [setConfirmationOpen]);
188189

packages/clerk-js/src/ui/contexts/components/Plans.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,14 @@ export function normalizeFormatted(formatted: string) {
3232

3333
const useBillingHookParams = () => {
3434
const subscriberType = useSubscriberTypeContext();
35-
const organizationCtx = useOrganizationContext();
3635
const allowBillingRoutes = useProtect(
3736
has =>
3837
has({
3938
permission: 'org:sys_billing:read',
4039
}) || has({ permission: 'org:sys_billing:manage' }),
4140
);
41+
// Do not use `useOrganization` to avoid triggering the in-app enable organizations prompt in development instance
42+
const organizationCtx = useOrganizationContext();
4243

4344
return {
4445
for: subscriberType,

0 commit comments

Comments
 (0)