Gallery (#1103)

* Header and flex box for gallery (#1102)

* Remove old gallery

* Setup gallery link

* header for gallery

* images flex box

* indentation corrected

* resolved issues

* changes part 3 lol

* resolved issue - favicon and description

* resolved css issue

* newline & semicolon resolved

Co-authored-by: Peter Kos <pkos91@icloud.com>
Co-authored-by: Saranya Dadi <sd1259@rit.edu>

* adding navbar

* newline at end of js

* fixes and refactoring js

* fixes

* Update sass/gallery.scss

Co-authored-by: Peter Kos <pkos91@icloud.com>

* Update sass/gallery.scss

Co-authored-by: Peter Kos <pkos91@icloud.com>

* Gallery Image Popup (#1107)

* adding html/css for modal

* making the modal open/close

* fixing stuff

* scss fixes

* fixes

* fixing the modal

* fixes

* Gallery Mobile Styling + CSS Cleanup (#1111)

* attempt 1 at cliping images

* mobile gallery + css cleanup

* fixes

* Dynamically adding images (#1115)

* first bit of changes

* attempts at dynamic

* FIx IAM credentials

* putting image into modal

* fixing image loading

* comment fix

* fixing my comment

Co-authored-by: Peter Kos <pkos91@icloud.com>

* trying to add lazy

* more attempts at Lazy loading

* Revert "more attempts at Lazy loading"

This reverts commit a61ba2f.

* Revert "trying to add lazy"

This reverts commit 22a9145.

* Lazy load background images in gallery (#1119)

* adding lazy

* fixing modal

* header information

* Dynamic loading + transitions!!!!!

* getting the last of the gallery content up

Co-authored-by: sjmiller7 <sj.miller.tn@gmail.com>

* Gallery thumbnail images and added logos (#1137)

* attempts at adding logos

* making thumbnails appear

* CD/local dev now uses .env for IAM auth; update README w/ local dev instructions (#1132)

* Add keys.js generation as part of GH actions

Would be ncie to refactor later to chain the build job

* switch from keys.js to env var

* Update readme to include IAM env settings

* Update build to use env var

* Remove TODO

* Env var CI/CD is now valid YAML :)

* Update gallery button w/ new icon + text

* fixing ios links issue

* Adjusting gallery modal design (#1151)

* adjusting modal design

* fixing modal scroll breakpoint & dumb padding

* Fix package lock sync from backmerge

Co-authored-by: SaranyaDadi <71734141+SaranyaDadi@users.noreply.github.com>
Co-authored-by: Saranya Dadi <sd1259@rit.edu>
Co-authored-by: sjmiller7 <sj.miller.tn@gmail.com>
Co-authored-by: sjmiller7 <56453920+sjmiller7@users.noreply.github.com>

Author: 4 people

.github/workflows/build.yml

@@ -22,3 +22,5 @@ jobs:
         node-version: '12'
     - run: npm ci
     - run: npm run build --if-present
+      env:
+        IDENTITY_POOL_ID: ${{ secrets.IDENTITY_POOL_ID }}

.github/workflows/deploy-production.yml

@@ -20,6 +20,8 @@ jobs:
       run: npm ci
     - name: Builder
       run: npm run build
+      env:
+        IDENTITY_POOL_ID: ${{ secrets.IDENTITY_POOL_ID }}
     - name: Deploy
       uses: peaceiris/actions-gh-pages@v3
       with:

.github/workflows/deploy-staging.yml

@@ -16,5 +16,7 @@ jobs:
       run: npm install
     - name: Build
       run: npm run build
+      env:
+        IDENTITY_POOL_ID: ${{ secrets.IDENTITY_POOL_ID }}
     - name: Ping FeaturePeek
       run: bash <(curl -s https://peek.run/ci)

README.md

@@ -35,6 +35,18 @@ $ npm run dev
 
 You should then be able to access the site at `localhost:1234`.
 
+### Environment variables
+
+The images in the [gallery](https://brickhack.io/gallery) are hosted on our AWS S3 instance.
+
+GitHub Actions (our CI/CD) uses a repository secret to store the environment variable.
+
+For local development on the gallery:
+- Create a `.env` file in the root of the project
+- Add `IDENTITY_POOL_ID="key"` to the top, where `key` is the AWS IAM key.
+
+**Note: Ask the Engineering team lead (@peterkos) for an AWS key if you do not have one.**
+
 # Development & Deployment
 
 All development work should be done locally in a new branch and/or fork. Then, make a pull request to have the code merged into the develop branch. Once the develop branch gets to a good state, it gets merged into the master branch for a production deployment.