Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Remove mongo database type. #94

Closed
wtgee opened this issue Feb 17, 2020 · 0 comments · Fixed by #97
Closed

Remove mongo database type. #94

wtgee opened this issue Feb 17, 2020 · 0 comments · Fixed by #97
Assignees
Labels
good first issue Good for newcomers

Comments

@wtgee
Copy link
Member

wtgee commented Feb 17, 2020

We have defaulted to using the simple file based database type for a while now because we don't actually use any of the mongo features on a unit and the install was proving to be difficult for a number of our builders.

The install could be mitigated by the docker migration but in general we probably just don't need to maintain this whole separate db type.

Need to remove the file, the tests, and references.

See panoptes/POCS#948

@wtgee wtgee added the good first issue Good for newcomers label Feb 17, 2020
@wtgee wtgee self-assigned this Feb 17, 2020
wtgee added a commit to wtgee/panoptes-utils that referenced this issue Feb 17, 2020
@wtgee wtgee added this to the v0.2.0 - The Cleanup milestone Feb 17, 2020
@wtgee wtgee closed this as completed in #97 Feb 17, 2020
wtgee added a commit that referenced this issue Feb 17, 2020
* Update license
* Contributing readme update
* Adding pycodestyle section
* Ignore vscode project dir
* Removing code that was duplicated in bayer.py
* Moving bayer into images subfolder
* Removing all google items from repo
* Removing mongo, closes #94
* Doc updates
* Reorganize social modules, closes #39
* Update the docker image used for testing
* Better coverage file
* Updating reqs with pip-compile
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant