Support Sqlite3 too

[adelowo]

Currently have a hard dependency on Postgres but it will be great to allow people quickly run this with sqlite

[adelowo]

Probably just switch out the database layer to sqlc or something

[oneEyedSunday]

I see that bun supports a sqlite dialect (https://github.com/uptrace/bun/tree/master/dialect/sqlitedialect)

I did a small poc here oneEyedSunday@4fe853a

I do think it may be unnecessary to flesh out the repository (e.g https://github.com/oneEyedSunday/sdump/blob/main/datastore/postgres/user.go) for sqllite since it has bun support.

Will run this end to end just to be sure.

[adelowo]

I see that bun supports a sqlite dialect (https://github.com/uptrace/bun/tree/master/dialect/sqlitedialect)

I did a small poc here oneEyedSunday@4fe853a

I do think it may be unnecessary to flesh out the repository (e.g https://github.com/oneEyedSunday/sdump/blob/main/datastore/postgres/user.go) for sqllite since it has bun support.

Will run this end to end just to be sure.

Looks good to me. Thanks for the work.

Happy to merge this in if you make a PR. Just a few comments I think I'd love to be fixed just from quickly scanning on mobile

[oneEyedSunday]

So I've made a PR on #37
I have to run it all locally and perhaps also with the migrations to see if its all ok.

[oneEyedSunday]

I've run this, and there's a little problem with the uuid extension on SQLite.

SQLite fails while running the fixtures, on

CREATE EXTENSION IF NOT EXISTS "uuid-ossp";

Installing the uuid extension on SQLite should let us keep the same fixtures.

Maybe this will help me setup locally https://github.com/manufaktor/articles/blob/main/using-uuid-in-sqlite.md

[adelowo]

True. SQLite doesn't have extensions. If this solution works then that's fine. We can use that.

I will take a look on Saturday at the PR. Apologies for late response

[oneEyedSunday]

True. SQLite doesn't have extensions. If this solution works then that's fine. We can use that.

I will take a look on Saturday at the PR. Apologies for late response

no worries man.

[adelowo]

fixed by #37