-
Notifications
You must be signed in to change notification settings - Fork 1
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
docs: Update Firebase Security Rules #116
Conversation
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.
looks good~
|
||
#### Registering Studies | ||
|
||
1. Navigate to your Firestore Database in the [Firebase console](https://console.firebase.google.com/) |
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.
I wonder if this is something that makes sense to do with a script? Like creating the collection and registering participants sounds fickle, and being able to automate that might be useful?
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.
Yeah that's a good idea, we have a CLI script for downloading data and this would be a neat addition to it. Most folks use Prolific at this point though so it won't be very high priority
Co-authored-by: Anna Murphy <[email protected]>
Co-authored-by: Anna Murphy <[email protected]>
Updates the documentation for the Firebase Security Rules to match the default rule and provides additional context for registering studies.
The API keys that I blurred out are technically public so there's no concern of the old photos that are unblurred but I figure it's generally safer to blur them anyways.