The app is using 4 different databases, instead it should use only one main database and 4 tables. about routinger HOT 10 CLOSED

Hello, this would be my first contribution. I was just wonder if I could take a stab at this for Hacktoberfest? Would I need to wait until it starts? Thank you!

from routinger.

The thing is SQFLite package used for database handling in this app isn't yet supported in web versions of flutter. So you need to run the app in an android or iOS emulator.

Thank you! I got it fixed, I'll make the PR in October.

from routinger.

Yes, surely, you can try it. I am assigning this to you. Unfortunately, if you want this to be counted towards hacktoberfest, wait to make the pull request till 1st October. You can fork the repo now do the required changes and then make a pull request on 1st October. That will save you a lot of time too.

from routinger.

@RepeaterCreeper You may need to wipe out your android emulator data and install the app yet again before building it and working on it. Because I have made some database changes which will generate errors if you don't do a fresh install or delete the saved data on the device

from routinger.

@RepeaterCreeper You may need to wipe out your android emulator data and install the app yet again before building it and working on it. Because I have made some database changes which will generate errors if you don't do a fresh install or delete the saved data on the device

Awesome. I'll keep that in mind.

from routinger.

Great.

from routinger.

Hey! So I was able to make it so it only uses 1 database with 4 tables. However I did notice an error that was popping up in my console about this:

At first I thought I broke something, so I reverted to the original codebase and it was providing the same error. I googled it and it wasn't providing me much help aside from a SO answer saying I needed to add something to the MainActivity.

from routinger.

The thing is SQFLite package used for database handling in this app isn't yet supported in web versions of flutter. So you need to run the app in an android or iOS emulator.

from routinger.

@RepeaterCreeper Please fetch upstream before submitting a PR and resolve conflicts. Also, if you have done it, please can you make a PR? We have to make some changes to the database. But if we do that now, it will show conflicts with your branch. We don't want that. Therefore, we have put a hold on it.

But we can't hold it for much longer.

Thank you

from routinger.

@RepeaterCreeper Please fetch upstream before submitting a PR and resolve conflicts. Also, if you have done it, please can you make a PR? We have to make some changes to the database. But if we do that now, it will show conflicts with your branch. We don't want that. Therefore, we have put a hold on it.

But we can't hold it for much longer.

Thank you

Alright, I'll do that.

from routinger.