search

feedhenry / fh-db

Apache License 2.0
1 stars 16 forks source link

flagged mongodb calls #38

Closed cfoskin closed 6 years ago

cfoskin commented 6 years ago

Flagged the db calls and Im not sure if this will be changed as it is a layer that sits on top of mongo

Ping @mikenairn @odra @PhilipGough

odra commented 6 years ago

I think it boils down to where fh-db is being used since it's just a mongodb wrapper/library.

cfoskin commented 6 years ago

@odra Agreed Do you think this one can be closed ?

odra commented 6 years ago

yep this close this one.

mikenairn commented 6 years ago

@cfoskin @odra Agreed. Changes, if any, will be required where this is being used rather in here. I'd say we can close this. Do we have an idea of what uses fh-db?

cfoskin commented 6 years ago

@mikenairn I know fh-mbaas-api uses this anyway https://github.com/feedhenry/fh-mbaas-api/search?utf8=%E2%9C%93&q=fh-db&type=