Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Migrate To-Do List Storage to SQLite #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sweep-ai[bot]
Copy link

@sweep-ai sweep-ai bot commented Jun 29, 2023

This PR addresses issue #8, which requested the migration of the to-do list storage from a text file to a SQLite database.

Here's a summary of the changes:

  • Created a new SQLite database file todo_db.sqlite to store the to-do list items.
  • Updated the append_to_do, delete_to_do, and get_to_do_list_from_file functions in todo.py to interact with the SQLite database instead of the text file.
  • Added exception handling to these functions to catch and handle any errors that occur while interacting with the database.

These changes should make the to-do list feature more robust and efficient.

Please review and let me know if any changes are needed.

Fixes #8.

To checkout this PR branch, run the following command in your terminal:

git checkout sweep/migrate-to-sqlite

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sweep: Migrate ohno/todo.py to sqlite from existing approach of .txt file
0 participants