Skip to content

Conversation

@morganchen12
Copy link
Contributor

No description provided.

@morganchen12 morganchen12 changed the title Mc/pipelines Add pipeline snippets Oct 29, 2025
Copy link

@MarkDuckworth MarkDuckworth left a comment

Choose a reason for hiding this comment

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

I did not finish the review, but here is some high level feedback that needs to be addressed. Some across all samples.

Copy link

@MarkDuckworth MarkDuckworth left a comment

Choose a reason for hiding this comment

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

Approved with comments. There are a few remaining errors and a few nits I commented on.

field("tags").arrayConcat([
field("pages").greaterThan(100)
.conditional(constant("longRead"), constant("shortRead"))
]).as("extendedTags")

Choose a reason for hiding this comment

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

I like this example!

}

async function strSplitFunction() {
// not yet supported until GA

Choose a reason for hiding this comment

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

it is supported now

Choose a reason for hiding this comment

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

in SDKs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wasn't able to find this method after updating both the web and iOS SDKs. I haven't checked the other ones yet.

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.

3 participants