Scott Trinh

Results 319 comments of Scott Trinh

Apologies for missing all of these pings (I'll try to get my notifications to be more noisy!) I'll take a deeper look into this today. Sounds like just the most...

We've talked about this one before, and I just wanted to note that CSV (probably TSV?) is a little tricky for non-table-like data. So we'd likely embed "interesting" data as...

I think I'm leaning toward 1: our client connections really do _want_ to try to read from the environment, so even if you don't plan on using environment variables to...

Yeah, great point. ["`never` is assignable to anything"](https://www.typescriptlang.org/docs/handbook/type-compatibility.html#any-unknown-object-void-undefined-null-and-never-assignability) Options: 1. Remove this key from the resulting object. 2. Assign `undefined` to this key in the resulting object and make the...

This is probably related to some `tsconfig.json` option on your side being different than the config we use and test. Can you provide some insight into your setup so we...

> so I don't know do we wan't to add support for these or should I return an error from the provider if user tries to set any of these....

> I believe for the replace part inside edgeql-js.ts file you had sth like this in mind, but I don't think we really need this, because I believe in the...

> I'm really not sure should the driver behave differently depending on the server version, I am not sure does the driver should return std::cal::local_date etc as the typename of...

@jaclarke Sorry about missing this: the built-in UI needed this update, too, so: [`d6c1053` (#1145)](https://github.com/edgedb/edgedb-js/pull/1145/commits/d6c1053766fb90d9a1d4187716bb00215dba7d32)

Going to put this back in draft, it's not nearly ready: - The core verify methods do _not_ verify emails without a pkce verifier. They should be updated. - We...