Completed the functionality in /checkin_register to properly insert a user into the user table in the SQLite database. I also added some error checking around it in case the original function raised any exceptions. The insert_user() function also had to be modified to allow for a status to be returned back to the command.
Issues
Closes #338
Type of change
Select one or more of the following:
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Other (describe below)
How Has This Been Tested?
[ ] Run the discord bot
[ ] Run the /checkin_register command
[ ] See the Dm and verify you were properly added to the database
[ ] Try to run the /checkin_register command again
[ ] See the error stating you are already in the database
Checklist:
[x] All local commits have been pushed to remote
[x] All changes on the base branch have been merged into this branch, either by rebase or merge
[x] My code is PEP-8 compliant (excluding maximum line length, keep that to 100ish characters)
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
Description
Completed the functionality in
/checkin_register
to properly insert a user into the user table in the SQLite database. I also added some error checking around it in case the original function raised any exceptions. Theinsert_user()
function also had to be modified to allow for a status to be returned back to the command.Issues
Closes #338
Type of change
Select one or more of the following:
How Has This Been Tested?
/checkin_register
command/checkin_register
command againChecklist: