search

griffithlab / civicpy

A python interface for the CIViC db application
MIT License
8 stars 5 forks source link

Status filters #41

Closed ahwagner closed 5 years ago

ahwagner commented 5 years ago

This draft PR is for evaluation of a status_filters parameter for the get_all_<element> methods. The intended usage is to add to the parameter list any status to exclude from results.

Discussion point: we could alternatively provide the complete list by default as a include_status parameter, and have people remove statuses from the list that they do not wish to view.

ahwagner commented 5 years ago

😆 hah, that's funny. I didn't realize this was already in civic-server. I actually didn't like this as-is; I'm going to refactor quick, and in a way that should make it easy to re-use in CIViC. Stay tuned.

susannasiebert commented 5 years ago

If we add an entrypoint to setup.py it will make it easier to call the script from a civicpy install.