search

pickardj79 / bga-galaxytrucker

Other
1 stars 0 forks source link

Hide all usages of DB to a helper #26

Open kuwizard opened 3 years ago

kuwizard commented 3 years ago

There's a great DB wrapper in BANG!, I would like to reintroduce it here. It really helps with standard operations. And also as a result of this ticket no raw SQL should be seen anywhere but inside this helper

demmerichs commented 3 years ago

After looking through all issues, I think this is one I can work on quite well. I will probably have time on wednesday to do this, I will start out by looking/copying the wrapper structure from BANG. However, if this is something you actually really want to do yourself, then of course i dont want to take it away from you :) Instead, I would start playing the current code state a few times to note a few more improvements and try to implement those :)

kuwizard commented 3 years ago

However, if this is something you actually really want to do yourself, then of course i dont want to take it away from you :)

I am perfectly fine with you doing this, thanks a lot!

demmerichs commented 3 years ago

As discussed, I will stop working on this for now. No Implementation happened till now, only reading and understanding of existing bang code.