Skip to content
This repository was archived by the owner on Oct 9, 2025. It is now read-only.

Conversation

@soedirgo
Copy link
Member

What kind of change does this PR introduce?

docs update

What is the current behavior?

defaultToNull only applies when doing bulk inserts/upserts, but this isn't reflected in the docs

What is the new behavior?

Update the docs

Additional context

#417 (comment)

Copy link
Member

@dshukertjr dshukertjr left a comment

Choose a reason for hiding this comment

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

The code looks good to me, but the failing test is okay?

@soedirgo soedirgo force-pushed the chore/docs-defaultToNull branch from 147bb3e to a2166f3 Compare July 21, 2023 07:59
@soedirgo
Copy link
Member Author

Whoops, had a bug - thank god for tests 😄

@soedirgo soedirgo force-pushed the chore/docs-defaultToNull branch from a2166f3 to 95c9bf1 Compare July 21, 2023 08:00
@soedirgo soedirgo merged commit 32e220c into master Jul 21, 2023
@soedirgo soedirgo deleted the chore/docs-defaultToNull branch July 21, 2023 08:03
@github-actions
Copy link

github-actions bot commented Aug 6, 2023

🎉 This PR is included in version 1.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants