-
Notifications
You must be signed in to change notification settings - Fork 12
v2.1.9 #291
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
v2.1.9 #291
Conversation
Hagellach37
commented
Feb 3, 2020
- adjust firebase rules for leaderboard Read access to v2/users in firebase #282 (this has also been merged into master already)
- fix slack message bug during project creation send success message from within try block #289 #290
- add script to archive old projects Delete/Archive old projects #280
…actionAbortedError
add script to archive old firebase projects
Revert "add script to archive old firebase projects"
Update db rule
send success message from within try block #289
fb_db = auth.firebaseDB() | ||
ref = fb_db.reference("projects") | ||
projects = ref.get() | ||
logger.info("got old projects from firebase") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe Retrieved {n_projects} MapSwipe v1 projects from firebase
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to be a migration rather than a script
that will be run multiple times. I suggest making that clearer (maybe move to scripts/migration/
?) and stating the end goal at the top of the migration file.