Skip to content
This repository has been archived by the owner on Mar 24, 2023. It is now read-only.

add save_dataframe() method to sqlite wrapper; remove Fetchers save() and load() methods #11

Closed
mageirakos opened this issue Jul 23, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@mageirakos
Copy link
Collaborator

Motivation

Upon review the save() and load() methods inside the Fetcher classes are redundant.

Modification

  1. Add a save_dataframe() method to our sqlite wrapper which in conjuntion with the get_dataframe() method
    achieves the same results.
  2. Modify classes and the fetch_docs/fetch_issues scripts to work as intended
@mageirakos mageirakos added the enhancement New feature or request label Jul 23, 2020
@mageirakos mageirakos self-assigned this Jul 23, 2020
@mageirakos mageirakos removed their assignment Sep 2, 2020
@bari12 bari12 closed this as completed Mar 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants