Merge branch 'develop' into toast-color-contrast-refactor

Author: raclim

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,8 +1,13 @@
-Fixes #issue-number
+### Context or Issue Number:
+<!-- Please add the issue this relates to if one is available. Otherwise please describe the current state of the codebase and what this PR attempts to fix. -->
 
-Changes:
+### Demo:
+<!-- Please add a screenshot for UI related changes -->
 
-I have verified that this pull request:
+### Changes:
+<!-- Summarise your changes -->
+
+### I have verified that this pull request:
 
 * [ ] has no linting errors (`npm run lint`)
 * [ ] has no test errors (`npm run test`)

contributor_docs/s3_configuration.md

@@ -21,7 +21,8 @@
     }
 ]
 ```
-3. In permissions, add the following bucket policy. Change "YOUR_BUCKET_NAME" to reflect name of the S3 bucket.
+3. Uncheck "Block all public access" under "Block public access (bucket settings)".
+4. In permissions, add the following bucket policy. Change "YOUR_BUCKET_NAME" to reflect name of the S3 bucket.
 ```
 {
 	"Version": "2008-10-17",
@@ -39,7 +40,6 @@
 	]
 }
 ```
-4. Uncheck "Block all public access" under "Block public access (bucket settings)".
 5. Under "Object Ownership", check "ACLs enabled" and set "Object Ownership" to "Object writer"
 6. Locate your AWS key and Secret Key. You can find this in the top AWS navigation under your name -> Security Credentials.
 7. Update the following lines to your .env file:
@@ -48,6 +48,7 @@ AWS_ACCESS_KEY={AWS_ACCESS_KEY}
 AWS_REGION={S3_BUCKET_REGION}
 AWS_SECRET_KEY={AWS_SECRET_KEY}
 S3_BUCKET={S3_BUCKET_NAME}
+S3_BUCKET_URL_BASE=https://{S3_BUCKET_NAME}.s3.{S3_BUCKET_REGION}.amazonaws.com/
 ```
 
 If your S3 bucket is in the US East (N Virginia) region (us-east-1), you'll

package-lock.json

@@ -177,7 +177,7 @@ passport.use(
           // with emails that are connected to the same GitHub account
           if (existingEmailUsers.length > 1) {
             existingEmailUser = existingEmailUsers.find(
-              (u) => (u.email = primaryEmail)
+              (u) => u.email === primaryEmail
             );
           } else {
             [existingEmailUser] = existingEmailUsers;