phlask / phlask-map

Code behind the Phlask Web Map
https://beta.phlask.me
34 stars 36 forks source link

Initial Database Cleanup #175

Closed gcardonag closed 3 years ago

gcardonag commented 3 years ago

Is your feature request related to a problem? Please describe. In order to ease onboarding and future development, we would like to deprecate our current Firebase databases and replace them with a standard set of databases to support all of our planned environments.

Describe the solution you'd like

Describe alternatives you've considered N/A

Additional context N/A

zeeatwork commented 3 years ago

I have not deleted phlask-web-map because it is currently the default database. Let me know if you'd like to assign one of the new databases as the default

zeeatwork commented 3 years ago

I also kept 'phlask-web-map-food-updated' because you've listed updated hours twice. I'm not sure if you meant to list 'updated hours' only or 'updated hours and 'food-updated'

zeeatwork commented 3 years ago

complete

gcardonag commented 3 years ago

Hey Zenzi, after reviewing in Firebase, it seems we cannot delete the phlask-web-map DB because it is tied to the Firebase project (hence the shared name). We can leave that one as-is but lock down its permissions so that it can't be read/written to.

Regarding the phlask-web-map-food-updated database, we can clean that one up as well since it is no longer referenced in our codebase. Thanks for checking!