-
Notifications
You must be signed in to change notification settings - Fork 418
fix(backend): Correct WaitlistEntryAPI list return type and export WaitlistEntry #7280
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(backend): Correct WaitlistEntryAPI list return type and export WaitlistEntry #7280
Conversation
🦋 Changeset detectedLatest commit: c2a2071 The changes in this PR will be included in the next version bump. This PR includes changesets to release 11 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughCorrects the TypeScript return type of Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Poem
Pre-merge checks and finishing touches✅ Passed checks (5 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (35)
🔇 Additional comments (1)
Comment |
| /** | ||
| * The email address associated with the `WaitlistResource`. | ||
| */ | ||
| emailAddress: string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file is the FAPI representation of a WaitlistEntry, which is just
{
"object": "waitlist_entry",
"id": "wle_35nCTK0zjvUt181oXWwlVvd3aTO"
}
I think the fields you've added here are from the BAPI representation? backend uses https://github.com/clerk/javascript/blob/69833fefe7a2f2eb0eb72d94617fe39a208ed310/packages/backend/src/api/resources/WaitlistEntry.ts which has these fields (albeit without the comments)
@clerk/agent-toolkit
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/dev-cli
@clerk/elements
@clerk/clerk-expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/clerk-react
@clerk/react-router
@clerk/remix
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/themes
@clerk/types
@clerk/upgrade
@clerk/vue
commit: |
578f72f to
5d52b88
Compare
…itlistEntry Fixes TypeScript return type for `WaitlistEntryAPI.list()` to properly reflect that it returns an array of `WaitlistEntry` objects within the paginated response, and exports the `WaitlistEntry` type from the backend package. ## Changes - Fix `WaitlistEntryAPI.list()` return type from `PaginatedResourceResponse<WaitlistEntry>` to `PaginatedResourceResponse<WaitlistEntry[]>` - Export `WaitlistEntry` type from `@clerk/backend` package index Signed-off-by: Kenton Duprey <kenton@clerk.dev>
5d52b88 to
dd3dd61
Compare
Description
Fixes TypeScript return type for
WaitlistEntryAPI.list()to properly reflect that it returns an array ofWaitlistEntryobjects within the paginated response, and exports theWaitlistEntrytype from the backend package.Changes
WaitlistEntryAPI.list()return type fromPaginatedResourceResponse<WaitlistEntry>toPaginatedResourceResponse<WaitlistEntry[]>WaitlistEntrytype from@clerk/backendpackage indexFixes USER-4021
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change
Summary by CodeRabbit
Bug Fixes
New Features
Chores
✏️ Tip: You can customize this high-level summary in your review settings.