Skip to content

Commit

Permalink
Merge pull request #104 from palantir/ssanjay/fixActionTypes
Browse files Browse the repository at this point in the history
Fix action params
  • Loading branch information
ssanjay1 authored Mar 5, 2024
2 parents 5cae8ae + c4b11cf commit 265ea37
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
5 changes: 5 additions & 0 deletions packages/client/changelog/@unreleased/pr-104.v2.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
type: fix
fix:
description: Fix action params
links:
- https://github.com/palantir/osdk-ts/pull/104
4 changes: 2 additions & 2 deletions packages/client/src/actions/Actions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ import type { Attachment } from "../object/Attachment.js";
import type { Osdk, OsdkObjectPrimaryKeyType } from "../OsdkObjectFrom.js";
import type { NOOP } from "../util/NOOP.js";
import type { NullableProps } from "../util/NullableProps.js";
import type { PartialByNotStrict } from "../util/PartialBy.js";
import type { PartialBy } from "../util/PartialBy.js";
import type { ActionReturnTypeForOptions } from "./applyAction.js";

export type ApplyActionOptions =
Expand Down Expand Up @@ -67,7 +67,7 @@ type NotOptionalParams<X extends ActionParametersDefinition> = {
export type OsdkActionParameters<
X extends ActionParametersDefinition,
> = NullableProps<X> extends never ? NotOptionalParams<X>
: PartialByNotStrict<NotOptionalParams<X>, NullableProps<X>>;
: PartialBy<NotOptionalParams<X>, NullableProps<X>>;

export type ActionSignatureFromDef<T extends ActionDefinition<any, any, any>> =
NonNullable<T["__OsdkActionType"]> extends never
Expand Down

0 comments on commit 265ea37

Please sign in to comment.