Multiple documentation issues (and one real) about nativescript-sqlite

[NathanaelA]

Hi Thanks for all the AWESOME feedback. :-)

const Sqlite = require( "@proplugins/nativescript-sqlite" );

This is a mistake; I was going to move sqlite to @proplugins namespace, and even had it fully working under that namespace, but then just before I moved it, I realized that it could break a lot of clients. So I reverted all those changes. Looking at the docs, it looks like I missed that one, I'll get that fixed. 😀

import {Sqlite} from "@proplugins/nativescript-sqlite";

Good catch, I totally glossed over that one.

const dbPromise = new Sqlite("MyTable");
const db = await dbPromise;

You can actually do, without the new: ;-)
const db = await Sqlite("MyTable");

I updated the docs. 😀

4. Also the official example

This was more for simplicity that I joined the two examples; but I can see your point so I turned it into two separate code snippets. 😀

5. Version, under a callback, it does return a number. However, looking at the code path I believe you are correct under a promise it will return the value array... Good catch, I've never noticed that discrepancy...

It will be fixed in the next version to be a number under both path's.

[emirotin]

JFYI I've checked it and the version is fixed indeed in 2.6.2.

const db = await Sqlite(dbName);
doesn't work though while
const db = await new Sqlite(dbName);
works fine (yet looks weird 😄)

[NathanaelA]

Weird, the await sqlite should be working ; I'll run a test... :)

Nathanael A.

[NathanaelA]

Should be working fine in 2.6.3, had a silly if order precedence bug...