Skip to content

Conversation

alex-all3dp
Copy link
Contributor

Closes 803

I also noticed that D1NextModeTagCache was missing a unit test, so I had Copilot draft one up. If you don't want that included, let me know and I'll drop that commit.

Copy link

changeset-bot bot commented Jul 21, 2025

🦋 Changeset detected

Latest commit: 4520da6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@opennextjs/cloudflare Patch

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

@alex-all3dp alex-all3dp changed the title Fix missing SQL parameter bindings in D1NextModeTagCache.getLastRevalidated() Fix missing SQL parameter bindings in D1NextModeTagCache.getLastRevalidated() Jul 21, 2025
@vicb
Copy link
Contributor

vicb commented Jul 22, 2025

Thanks @alex-all3dp
Could you please add a patch changeset (via pnpm changeset)

Copy link

pkg-pr-new bot commented Jul 22, 2025

Open in StackBlitz

npm i https://pkg.pr.new/@opennextjs/cloudflare@804

commit: 4520da6

@alex-all3dp
Copy link
Contributor Author

Thanks @alex-all3dp Could you please add a patch changeset (via pnpm changeset)

Done ✅

Copy link
Contributor

@vicb vicb left a comment

Choose a reason for hiding this comment

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

Thanks!!

@vicb vicb merged commit 72d8d5b into opennextjs:main Jul 22, 2025
7 checks passed
@alex-all3dp alex-all3dp deleted the alex-all3dp/fix/missing-sql-param-bindings branch July 22, 2025 08:26
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.

[BUG] D1_ERROR: Wrong number of parameter bindings for SQL query.

2 participants