jepiqueau / react-sqlite-app-starter

Ionic/React SQLite Application Starter
MIT License
22 stars 14 forks source link

fix: build failed, lost jeep-sqlite package #25

Closed sabakugaara closed 2 years ago

jepiqueau commented 2 years ago

@sabakugaara This is not required, delete your node_modules folder and the package-lock.json and do npm install. it should work. jeep-sqlite is loaded in node_modules as a dependency of @capacitor-community/sqlite

sabakugaara commented 2 years ago

OK, I use pnpm and manually install it, otherwise ionic will fail to build.

jepiqueau commented 2 years ago

@sabakugaara i tested with npm and yarn and it works, i also tested is with pnpm and here you right but you must also pnpm i --save sql.js otherwise it will not work. So i will not accept to merge the PR but i will modify the doc of capacitor-community/sqlite to add for pnpm these two supplementary installs. Do you agree with that?

sabakugaara commented 2 years ago

Thank you! I agree.