Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Nov 6, 2025

Auto-generated PR to update Typedoc documentation from clerk/javascript@9abb76f

@github-actions github-actions bot requested a review from a team as a code owner November 6, 2025 18:30
@vercel
Copy link

vercel bot commented Nov 6, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
clerk-docs Ready Ready Preview Nov 6, 2025 10:27pm

@SarahSoutoul
Copy link
Contributor

plan of action:
closing this.
lets get clerk/javascript#7158 merged first
then trigger new CI action
then merge #2756 into that newly created PR from the cI action

| ---------------------------------- | -------------------------------------------------------------------------------------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| <a id="debug"></a> `debug` | <code>() => \{ [key: string]: any; \}</code> | Used to help debug issues when using Clerk in development. |
| <a id="gettoken"></a> `getToken` | <code>() => Promise\<null\> \| () => Promise\<string\> \| ServerGetToken \| () => Promise\<null\></code> | A function that gets the current user's [session token](/docs/guides/sessions/session-tokens) or a [custom JWT template](/docs/guides/sessions/jwt-templates). |
| <a id="has"></a> `has` | <code>CheckAuthorizationFromSessionClaims \| () => false</code> | A function that checks if the user has an organization role or custom permission. |
Copy link
Contributor

Choose a reason for hiding this comment

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

Lost a link here

Copy link
Contributor

Choose a reason for hiding this comment

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

Having a look rn

Copy link
Contributor

Choose a reason for hiding this comment

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

These links were broken in the first place, as there aren't links in the first place to link these too. This is also a typedoc that's not being used as we discussed. So I think we're safe to ignore.

| <a id="pastdueat"></a> `pastDueAt` | <code>null \| number</code> | Unix timestamp (milliseconds) of when the subscription became past due. |
| <a id="payerid"></a> `payerId` | `string` | The ID of the payer for this subscription. |
| <a id="status"></a> `status` | <code>"abandoned" \| "active" \| "ended" \| "canceled" \| "incomplete" \| "past_due"</code> | The current status of the subscription. |
| <a id="subscriptionitems"></a> `subscriptionItems` | <code>[BillingSubscriptionItem](billing-subscription-item.mdx)[]</code> | Array of subscription items in this subscription. |
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a broken link / missing a link replacement

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure of this one bcs of several reasons:

  1. It's not being used as a typedoc.
  2. It's only being used in this typedoc.
  3. There is no existing link to link this to.

| `props` | [`Without`](../types/without.mdx)\<`WithClerkProp`\<`PropsWithChildren`\<[`PlanDetailsButtonProps`](plan-details-button-props.mdx)\>\>, `"clerk"`\> |
| Parameter | Type |
| --------- | --------------------------------------------------------------------------------------------------------------------------- |
| `props` | `Without`\<`WithClerkProp`\<`PropsWithChildren`\<[`PlanDetailsButtonProps`](plan-details-button-props.mdx)\>\>, `"clerk"`\> |
Copy link
Contributor

Choose a reason for hiding this comment

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

broken link to plan details button props

Copy link
Contributor

Choose a reason for hiding this comment

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

This typedoc isn't used

Comment on lines +7 to +11
| `params?` | `T$1` |

## Returns

[`UseOrganizationListReturn`](../shared/use-organization-list-return.mdx)\<`T`\>
`UseOrganizationListReturn`\<`T$1`\>
Copy link
Contributor

Choose a reason for hiding this comment

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

No idea whats going on with the $1 getting added

Copy link
Contributor

Choose a reason for hiding this comment

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

Also not used as a typedoc but yeah weird behaviour.

## Returns

[`SignInSignalValue`](../types/sign-in-signal-value.mdx)
`SignInSignalValue`
Copy link
Contributor

Choose a reason for hiding this comment

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

Lost broken link here

Copy link
Contributor

Choose a reason for hiding this comment

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

This typedoc isn't used, and doesn't have a link to lead to.

@SarahSoutoul
Copy link
Contributor

SarahSoutoul commented Nov 6, 2025

Thanks for all the catches @NWylynko - have responded to them all, and as we discussed, it's looking like either the broken links are on typedocs that aren't used in the docs at all or the links were already broken and there is no real link to lead them to, so imo, we could prob ignore these.

I am going to add the changes of this PR in here, so I can test everything. @alexisintech

@SarahSoutoul
Copy link
Contributor

Update: Noticed one issue caused by this catchReplacement pattern.

Screenshot 2025-11-06 at 4 43 39 pm

Basically, it gets applied to the BillingMoneyAmount text within the BillingMoneyAmount Javascript type here. That text shouldn't be linked since we're already on the BillingMoneyAmount page (see the screenshots - local vs live docs). If I remove it, it unlinks all the BillingMoneyAmount in the backend typedocs, which aren't used anyway. All the other places where BillingMoneyAmount is rendered, it is linked properly since it's inside the shared package.

Screenshot 2025-11-06 at 4 43 00 pm Screenshot 2025-11-06 at 4 42 07 pm

So I say we prob want to remove this catch replacement but wanna get everyone's thoughts @alexisintech @NWylynko.

@alexisintech
Copy link
Member

Update: Noticed one issue caused by this catchReplacement pattern.

Screenshot 2025-11-06 at 4 43 39 pm Basically, it gets applied to the `BillingMoneyAmount` text within the `BillingMoneyAmount` Javascript type [here](https://clerk.com/docs/reference/javascript/types/billing-money-amount). That text shouldn't be linked since we're already on the `BillingMoneyAmount` page (see the screenshots - local vs live docs). If I remove it, it unlinks all the `BillingMoneyAmount` in the `backend` typedocs, which aren't used anyway. All the other places where `BillingMoneyAmount` is rendered, it is linked properly since it's inside the `shared` package.

Screenshot 2025-11-06 at 4 43 00 pm Screenshot 2025-11-06 at 4 42 07 pm
So I say we prob want to remove this catch replacement but wanna get everyone's thoughts @alexisintech @NWylynko.

oh yeah then lets remove it !

@SarahSoutoul
Copy link
Contributor

Update: Noticed one issue caused by this catchReplacement pattern.
Screenshot 2025-11-06 at 4 43 39 pm
Basically, it gets applied to the BillingMoneyAmount text within the BillingMoneyAmount Javascript type here. That text shouldn't be linked since we're already on the BillingMoneyAmount page (see the screenshots - local vs live docs). If I remove it, it unlinks all the BillingMoneyAmount in the backend typedocs, which aren't used anyway. All the other places where BillingMoneyAmount is rendered, it is linked properly since it's inside the shared package.
Screenshot 2025-11-06 at 4 43 00 pm Screenshot 2025-11-06 at 4 42 07 pm
So I say we prob want to remove this catch replacement but wanna get everyone's thoughts @alexisintech @NWylynko.

oh yeah then lets remove it !

Okay, we'll need another JS PR then. And re-do the following process:

Screenshot 2025-11-06 at 5 56 37 pm

Will tackle it tomorrow!

@SarahSoutoul
Copy link
Contributor

C

Update: Noticed one issue caused by this catchReplacement pattern.

Screenshot 2025-11-06 at 4 43 39 pm Basically, it gets applied to the `BillingMoneyAmount` text within the `BillingMoneyAmount` Javascript type [here](https://clerk.com/docs/reference/javascript/types/billing-money-amount). That text shouldn't be linked since we're already on the `BillingMoneyAmount` page (see the screenshots - local vs live docs). If I remove it, it unlinks all the `BillingMoneyAmount` in the `backend` typedocs, which aren't used anyway. All the other places where `BillingMoneyAmount` is rendered, it is linked properly since it's inside the `shared` package.

Screenshot 2025-11-06 at 4 43 00 pm Screenshot 2025-11-06 at 4 42 07 pm
So I say we prob want to remove this catch replacement but wanna get everyone's thoughts @alexisintech @NWylynko.

@alexisintech @NWylynko Fixed this with this PR: clerk/javascript#7176 and newly generated Typedoc CI PR is here: #2776. So closing this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants