leogomezz4t / PyTextNow_API

Python texting API utilizing TextNow API
MIT License
67 stars 41 forks source link

Create Database Framework #24

Closed SilverStrings024 closed 3 years ago

SilverStrings024 commented 3 years ago

Created database framework to replace user_sids.json to prevent the need for sudo NOTE: It's likely that, if installed globally, the program will need sudo to create the database.. TODO: Create dedicated script to create database that must be run as sudo once (unless package is uninstalled or removed).

SilverStrings024 commented 3 years ago
1. Delete the README.md from the pull requests because the one already in 1.1.8-test is ahead of this one.

2. You can delete the if **name** == "**main**" import in TNAPI.py. It was just for debugging.

3. Remove 'self' from all Container objects instantiations.

I thought you said you took out all those references?

leogomezz4t commented 3 years ago
1. Delete the README.md from the pull requests because the one already in 1.1.8-test is ahead of this one.

2. You can delete the if **name** == "**main**" import in TNAPI.py. It was just for debugging.

3. Remove 'self' from all Container objects instantiations.

I thought you said you took out all those references?

I did but your commit had some old code

leogomezz4t commented 3 years ago

Doesn't matter I already fixed TNAPI.py

leogomezz4t commented 3 years ago

noticed user_sid.json had data.. deleted it

thanks

leogomezz4t commented 3 years ago

I guess now we have to test it

bryanperez43 commented 3 years ago

we need

  1. integrate the database
  2. test / fix any bugs (if any)
  3. re-write example scripts
  4. edit the README.md examples
SilverStrings024 commented 3 years ago

Don't forget also we have things that need tested listed in the integration channel on the discord. We still need to figure out who is doing what so we don't work on the same stuff.

SilverStrings024 commented 3 years ago

Merging

leogomezz4t commented 3 years ago

ok