- 
                Notifications
    You must be signed in to change notification settings 
- Fork 403
chore(types): Drop deprecations #2109
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
Conversation
| 🦋 Changeset detectedLatest commit: eb4a21e 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 | 
fe363d9    to
    e6c457f      
    Compare
  
    769e1c3    to
    89d9c98      
    Compare
  
    e6c457f    to
    cb0f046      
    Compare
  
    89d9c98    to
    48c87b1      
    Compare
  
    b05817b    to
    937338d      
    Compare
  
    cb0f046    to
    2970cba      
    Compare
  
    937338d    to
    e013129      
    Compare
  
    4cbb4c9    to
    55612d2      
    Compare
  
    55612d2    to
    f052495      
    Compare
  
    e013129    to
    fd01ed2      
    Compare
  
    fd01ed2    to
    e0107dd      
    Compare
  
    e0107dd    to
    c82f3ae      
    Compare
  
    2961583    to
    6e7e3ec      
    Compare
  
    6e7e3ec    to
    acafd5b      
    Compare
  
    acafd5b    to
    418ddc0      
    Compare
  
    | isSatellite: props.isSatellite || isTruthy(process.env.NEXT_PUBLIC_CLERK_IS_SATELLITE), | ||
| proxyUrl: props.proxyUrl || (process.env.NEXT_PUBLIC_CLERK_PROXY_URL as any), // added as any to cater issue with type resolution | ||
| domain: props.domain || (process.env.NEXT_PUBLIC_CLERK_DOMAIN as any), // added as any to cater issue with type resolution | ||
| isSatellite: props.isSatellite || (isTruthy(process.env.NEXT_PUBLIC_CLERK_IS_SATELLITE) as any), // added as any to cater issue with type resolution | 
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.
@panteliselef i had to do this in order for the MultidomainAndOrProxy types to pass. let's discuss it tomorrow and see if we can simplify the type or fix the current issues.
418ddc0    to
    d5e3377      
    Compare
  
    | Moved localization related PRs to #2151 | 
d5e3377    to
    cc78b25      
    Compare
  
    …Object` `AuthObject` is a kind of superset of `ServerSideAuth`. The only difference is that instead of `auth.claims` in `AuthObject` you need to use `auth.sessionClaims`
…oviderOptionsWrapper
cc78b25    to
    eb4a21e      
    Compare
  
    
Description
Drop deprecations. Migration steps:
orgsjwt claim from session tokenauthparam ofwithServerAuth()callback toAuthObjectfromServerSideAuthingatsby-clerk-pluginauth.sessionClaimsinstead ofauth.claimsAuthObjectproperties fromauthpublishableKeyinstead offrontendApiClerkProviderOptionsWrappertype instead ofIsomorphicClerkOptionsChecklist
npm testruns as expected.npm run buildruns as expected.Type of change
Packages affected
@clerk/backend@clerk/chrome-extension@clerk/clerk-js@clerk/clerk-expo@clerk/fastifygatsby-plugin-clerk@clerk/localizations@clerk/nextjs@clerk/clerk-react@clerk/remix@clerk/clerk-sdk-node@clerk/shared@clerk/themes@clerk/typesbuild/tooling/chore