Cloudflare Developers•3y ago

i am using 3.5.0, local dev is ok ,something wrong in pages

akazwzOP•8/9/23, 12:22 PM

maybe, i have another project deployed some days ago, it works

Ggeelen it's late here now but I'll jump on and investigate tomorrow, just curious if it...

Matt Silverlock•8/9/23, 12:27 PM

Let me repro on my Remix project this morning when I’m in

Aakazwz i am using 3.5.0, local dev is ok ,something wrong in pages

Matt Silverlock•8/9/23, 12:27 PM

Try npx wrangler@3.4 <your commands>npx wrangler@3.4 <your commands> and report back

AArthie I'm also getting this error on 3.5.0: `this.client.prepare is not a function` do...

Matt Silverlock•8/9/23, 12:28 PM

In Pages or a Worker? Please share the code that triggers it.

AArthie Looking in my server.js **Worker** source code this is were it fails: ```ts /// ...

Matt Silverlock•8/9/23, 12:41 PM

What is client? A D1DatabaseD1Database

AArthie My database name is still prefixed with `env.__D1_BETA__db` looks like this migh...

Matt Silverlock•8/9/23, 12:45 PM

Right - 3.5.0 moved the binding code to our side. If you haven’t done anything custom (like copying code out of wrangler) then it should have been seamless

AArthie yes D1 beta database (latest beta not the legacy alpha)

Matt Silverlock•8/9/23, 12:46 PM

Can you help me understand how you got here - e.g. you upgraded to wrangler 3.5.0 and ran wrangler deploywrangler deploy

? Any other changes?

AArthie Ok was able to fix `this.client.prepare is not a function` on 3.5.0: 1) edit `w...

Matt Silverlock•8/9/23, 1:15 PM

How did you get that prefix? Did you copy (export) a Worker from the dash?

Matt Silverlock•8/9/23, 1:16 PM

For the record, I just upgraded to 3.5.0 to do this, and re-deployed my test D1 project at https://d1.silverlock.workers.dev/wnam/orders/select/

no issues.

AArthie Been using D1 since first public alpha, prefixing your d1 database binding with ...

Matt Silverlock•8/9/23, 1:23 PM

Ah, way back. We haven't required that in a very, very long time.

Adi•8/9/23, 1:24 PM

Quick question, are there any issues happening in d1 right now (legacy alpha db)? I just noticed a bunch of errors on my app - this.client.prepare is not a functionthis.client.prepare is not a function

HHasi but running a code like this ```js const result = req.context.db.prepare("SELEC...

Hasi•8/9/23, 1:24 PM

umm, sorry, but can someone take a look at this?

AAdi Quick question, are there any issues happening in d1 right now (legacy alpha db)...

Matt Silverlock•8/9/23, 1:24 PM

Scroll immediately up.

Adi•8/9/23, 1:25 PM

Oh whoops, sorry my eyes glanced over that...

HHasi umm, *sorry*, but can someone take a look at this?

Matt Silverlock•8/9/23, 1:25 PM

You're definitely hitting the wrong DB. Can you please share what

wrangler

wrangler

commands you're invoking? Here's a working Remix demo app: https://github.com/elithrar/remix-plus-d1

GitHub

GitHub - elithrar/remix-plus-d1: An example app for combining Cloud...

An example app for combining Cloudflare's D1 database with Remix loader functions. - GitHub - elithrar/remix-plus-d1: An example app for combining Cloudflare's D1 database with Remi...

HHasi umm, *sorry*, but can someone take a look at this?

Matt Silverlock•8/9/23, 1:25 PM

Also: https://developers.cloudflare.com/d1/examples/d1-and-remix/

Query D1 from Remix · Cloudflare D1 docs

Remix is a full-stack web framework that operates on both client and server. You can query your D1 database(s) easily from Remix with Remix’s data …

MMatt Silverlock You're definitely hitting the wrong DB. Can you please share what `wrangler` com...

Hasi•8/9/23, 1:26 PM

Basically wrangler d1 execute adoptdb

Hasi•8/9/23, 1:26 PM

I have only 1 db

HHasi Basically wrangler d1 execute adoptdb

Matt Silverlock•8/9/23, 1:31 PM

Is that it - exactly? No --local--local flag? Please be precise

MMatt Silverlock Is that it - exactly? No `--local` flag? Please be precise 🙂

Hasi•8/9/23, 1:33 PM

sorry I used local of course

Hasi•8/9/23, 1:33 PM

I both uploaded to local and prod

Hasi•8/9/23, 1:33 PM

the shema

Hasi•8/9/23, 1:33 PM

and checked that they present there

AArthie Ok was able to fix `this.client.prepare is not a function` on 3.5.0: 1) edit `w...

Adi•8/9/23, 1:35 PM

Can confirm this worked for me too

As Arthie mentioned, this was necessary when D1 first came out, I too had the binding setup ever since then, never updated it. Issue for ref - https://github.com/cloudflare/workers-sdk/issues/2402

HHasi *sorry* I used local of course

Matt Silverlock•8/9/23, 1:35 PM

See the steps here: d1-database

AAdi Can confirm this worked for me too 👍 As Arthie mentioned, this was necessary w...

Matt Silverlock•8/9/23, 1:36 PM

https://github.com/cloudflare/cloudflare-docs/pull/10298

changelog entry here

GitHub

[D1] Update changelog.md by elithrar · Pull Request #10298 · cloudf...

This PR adds a new changelog entry that:

highlights the new dash binding support
recommends users upgrade to wrangler 3.5.0
provides migration advice for legacy alpha users with D1_BETA prefixes

MMatt Silverlock See the steps here: https://discord.com/channels/595317990191398933/992060581832...

Hasi•8/9/23, 2:15 PM

mhm did it

Hasi•8/9/23, 2:15 PM

still same error

MMatt Silverlock Try `npx wrangler@3.4 <your commands>` and report back

akazwzOP•8/9/23, 2:48 PM

3.4 works

MMatt Silverlock For the record, I _just_ upgraded to 3.5.0 to do this, and re-deployed my test D...

akazwzOP•8/9/23, 2:55 PM

i can get this prefix too, in toml and dashboard, i set DB binding, but when i log the env, there is a prefix __D1BETA , i use wrangler@3.4 solved problem

akazwzOP•8/9/23, 3:03 PM

https://github.com/cloudflare/workers-sdk/commit/c302bec639c0eec10d07d6b950c0a2d3e16eab1e#diff-80a0404f59f781e58a6e6892e5457ca6bd1c7b7e5687155159bde57f6cc65452

GitHub

Removing D1 Shim (#3595) · cloudflare/workers-sdk@c302bec

Tried to remove all traces of the D1 shim
Fixing lint errors
Fixing deploy & middleware tests
Changing tests to expect absence of D1 shim, not presence
Adding changese...

chientrm.com•8/10/23, 1:41 AM

can you make wrangler D1 client interactive like other terminal sql client?

chientrm.com•8/10/23, 1:41 AM

I don't want to go to dash web everytime to run query.

Cyb3r-Jak3•8/10/23, 1:50 AM

Can run a query through wrangler with wrangler d1 execute <database name> --command <SQL here>wrangler d1 execute <database name> --command <SQL here>

CCyb3r-Jak3 Can run a query through wrangler with `wrangler d1 execute <database name> --com...

chientrm.com•8/10/23, 1:51 AM

I know but an interactive-like would be better.

chientrm.com•8/10/23, 1:52 AM

although it doesn't have to be a stateful connection.

chientrm.com•8/10/23, 1:52 AM

wrangler connect <database name>wrangler connect <database name>

Cchientrm.com I know but an interactive-like would be better.

James•8/10/23, 1:56 AM

There’s an awesome community project from @Isaac McFadyen | YYZ01 which might interest you: https://github.com/isaac-mcfadyen/d1-console

GitHub

GitHub - isaac-mcfadyen/d1-console: A full query console for Cloudf...

A full query console for Cloudflare's D1 database product. - GitHub - isaac-mcfadyen/d1-console: A full query console for Cloudflare's D1 database product.

i am using 3.5.0, local dev is ok ,something wrong in pages

Similar Threads

Similar Threads

Similar Threads