Skip to content

Commit 192fa72

Browse files
Configurable props must have a name (#72)
* Mark the `name` field of configurable props as required * Rename and export the `PipedreamClient` args --------- Co-authored-by: fern-api <115122769+fern-api[bot]@users.noreply.github.com> Co-authored-by: Jay Vercellone <[email protected]>
1 parent 277f851 commit 192fa72

16 files changed

+44
-18
lines changed

MIGRATE.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -835,7 +835,7 @@ incrementally without breaking your existing codebase. To do this, you can
835835
install the new SDK with an alias:
836836

837837
```bash
838-
npm install @pipedream/sdk-v2@npm:@pipedream/sdk@^2.0.0-rc.5 --save
838+
npm install @pipedream/sdk-v2@npm:@pipedream/sdk@^2.0.0-rc.6 --save
839839
```
840840

841841
Then, in your code, you can import the new SDK with the alias:

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@pipedream/sdk",
3-
"version": "2.0.0-rc.5",
3+
"version": "2.0.0-rc.6",
44
"private": false,
55
"repository": "github:PipedreamHQ/pipedream-sdk-typescript",
66
"type": "commonjs",

src/Client.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,8 @@ export class PipedreamClient {
7373
"x-pd-environment": _options?.projectEnvironment,
7474
"X-Fern-Language": "JavaScript",
7575
"X-Fern-SDK-Name": "@pipedream/sdk",
76-
"X-Fern-SDK-Version": "2.0.0-rc.5",
77-
"User-Agent": "@pipedream/sdk/2.0.0-rc.5",
76+
"X-Fern-SDK-Version": "2.0.0-rc.6",
77+
"User-Agent": "@pipedream/sdk/2.0.0-rc.6",
7878
"X-Fern-Runtime": core.RUNTIME.type,
7979
"X-Fern-Runtime-Version": core.RUNTIME.version,
8080
},

src/api/types/ConfigurableProp.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
*/
88
export interface ConfigurableProp {
99
/** When building `configuredProps`, make sure to use this field as the key when setting the prop value */
10-
name?: string;
10+
name: string;
1111
type?: string;
1212
/** Value to use as an input label. In cases where `type` is "app", should load the app via `getApp`, etc. and show `app.name` instead. */
1313
label?: string;

src/api/types/ConfigurablePropAlert.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ export interface ConfigurablePropAlert {
99
/** The content of the alert, which can include HTML or plain text. */
1010
content?: string;
1111
/** When building `configuredProps`, make sure to use this field as the key when setting the prop value */
12-
name?: string;
12+
name: string;
1313
/** Value to use as an input label. In cases where `type` is "app", should load the app via `getApp`, etc. and show `app.name` instead. */
1414
label?: string;
1515
/** A description of the prop, shown to the user when configuring the component. */

src/api/types/ConfigurablePropAny.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
export interface ConfigurablePropAny {
66
type?: "any";
77
/** When building `configuredProps`, make sure to use this field as the key when setting the prop value */
8-
name?: string;
8+
name: string;
99
/** Value to use as an input label. In cases where `type` is "app", should load the app via `getApp`, etc. and show `app.name` instead. */
1010
label?: string;
1111
/** A description of the prop, shown to the user when configuring the component. */

src/api/types/ConfigurablePropApp.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ export interface ConfigurablePropApp {
77
/** The name slug of the app, e.g. 'github', 'slack', etc. This is used to identify the app for which the account is being configured. */
88
app?: string;
99
/** When building `configuredProps`, make sure to use this field as the key when setting the prop value */
10-
name?: string;
10+
name: string;
1111
/** Value to use as an input label. In cases where `type` is "app", should load the app via `getApp`, etc. and show `app.name` instead. */
1212
label?: string;
1313
/** A description of the prop, shown to the user when configuring the component. */

src/api/types/ConfigurablePropBoolean.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
export interface ConfigurablePropBoolean {
66
type?: "boolean";
77
/** When building `configuredProps`, make sure to use this field as the key when setting the prop value */
8-
name?: string;
8+
name: string;
99
/** Value to use as an input label. In cases where `type` is "app", should load the app via `getApp`, etc. and show `app.name` instead. */
1010
label?: string;
1111
/** A description of the prop, shown to the user when configuring the component. */

src/api/types/ConfigurablePropDiscord.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
export interface ConfigurablePropDiscord {
66
type?: "$.discord.channel";
77
/** When building `configuredProps`, make sure to use this field as the key when setting the prop value */
8-
name?: string;
8+
name: string;
99
/** Value to use as an input label. In cases where `type` is "app", should load the app via `getApp`, etc. and show `app.name` instead. */
1010
label?: string;
1111
/** A description of the prop, shown to the user when configuring the component. */

src/api/types/ConfigurablePropInteger.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ export interface ConfigurablePropInteger {
99
/** The maximum value for this integer prop. */
1010
max?: number;
1111
/** When building `configuredProps`, make sure to use this field as the key when setting the prop value */
12-
name?: string;
12+
name: string;
1313
/** Value to use as an input label. In cases where `type` is "app", should load the app via `getApp`, etc. and show `app.name` instead. */
1414
label?: string;
1515
/** A description of the prop, shown to the user when configuring the component. */

0 commit comments

Comments
 (0)