From c4cebc35a3025d027bcc98de6cdef2e7f651f621 Mon Sep 17 00:00:00 2001
From: Jacob <jacob.schulzetenberg@gmail.com>
Date: Thu, 15 Sep 2022 20:38:18 -0500
Subject: [PATCH] Add parks images to cloudinary

---
 .prettierignore                               |  1 +
 controllers/app-config.js                     |  8 ++++-
 .../src/pages/app-settings/app-settings.tsx   |  5 ++-
 .../src/pages/app-settings/parks-settings.tsx | 16 +++++++---
 models/app-config.js                          |  2 --
 models/parks-model.js                         | 28 ++++++++++-------
 nodejs/goodreads.js                           |  1 +
 nodejs/instagram.js                           |  1 +
 nodejs/music.js                               |  1 +
 nodejs/parks.js                               | 31 +++++++++++++------
 nodejs/player-fm.js                           |  1 +
 11 files changed, 65 insertions(+), 30 deletions(-)

diff --git a/.prettierignore b/.prettierignore
index f71737da..15976c07 100644
--- a/.prettierignore
+++ b/.prettierignore
@@ -1,3 +1,4 @@
 node_modules
 package.json
 package-lock.json
+frontend
\ No newline at end of file
diff --git a/controllers/app-config.js b/controllers/app-config.js
index 79badb45..13369add 100644
--- a/controllers/app-config.js
+++ b/controllers/app-config.js
@@ -11,6 +11,7 @@ const mongoUtils = require('../nodejs/mongo-utils');
 const scheduler = require('../nodejs/scheduler');
 const { agenda } = require('../nodejs/agenda');
 const parksList = require('../config/parks');
+const parks = require('../nodejs/parks');
 const statesList = require('../config/states');
 const countriesList = require('../config/countries');
 const appModules = require('../nodejs/app-modules');
@@ -73,13 +74,18 @@ exports.getConfig = async (req, res) => {
  * POST /app-config
  * Save application config
  */
-exports.saveConfig = (req, res, next) => {
+exports.saveConfig = async (req, res, next) => {
   const data = req.body;
 
   if (!data) {
     return next('No config data to save');
   }
 
+  if (data.appName === 'parks') {
+    // eslint-disable-next-line no-underscore-dangle
+    await parks.save(req.user._id);
+  }
+
   if (data.parks && data.parks.visited) {
     data.parks.visited = data.parks.visited.map((x) => x.value);
   }
diff --git a/frontend/src/pages/app-settings/app-settings.tsx b/frontend/src/pages/app-settings/app-settings.tsx
index 1a469d92..3d40c1de 100644
--- a/frontend/src/pages/app-settings/app-settings.tsx
+++ b/frontend/src/pages/app-settings/app-settings.tsx
@@ -72,7 +72,10 @@ const AppSettings: React.FC = () => {
     setSaveSuccess(false);
 
     try {
-      const { data: response }: ServerResponse = await Request.post({ url: '/app-config/config', body });
+      const { data: response }: ServerResponse = await Request.post({
+        url: '/app-config/config',
+        body: { ...body, appName },
+      });
       setData(response.data);
       setSaveSuccess(true);
     } catch (e: any) {
diff --git a/frontend/src/pages/app-settings/parks-settings.tsx b/frontend/src/pages/app-settings/parks-settings.tsx
index f1b496a5..2a155b4f 100644
--- a/frontend/src/pages/app-settings/parks-settings.tsx
+++ b/frontend/src/pages/app-settings/parks-settings.tsx
@@ -1,16 +1,21 @@
 import React, { useEffect, useState } from 'react';
 import { useForm } from 'react-hook-form';
+import { makeStyles } from '@material-ui/core/styles';
 
-import { Button, Form, MultiSelect2 } from '@schulzetenberg/component-library';
+import { Button, Form, MultiSelect2, SwitchForm2 } from '@schulzetenberg/component-library';
+
+const useStyles = makeStyles(() => ({
+  textCenter: { textAlign: 'center' },
+}));
 
 type FormData = {
   visited: { value: string; label: string }[];
   options: { value: string; label: string }[];
-  schedule: string;
   cloudinaryUpload: boolean;
 };
 
 const ParksSettings: React.FC<{ data: FormData; isLoading: boolean; submit: any }> = ({ data, isLoading, submit }) => {
+  const classes = useStyles();
   const [options, setOptions] = useState<{ value: string; label: string }[]>([]);
 
   const {
@@ -26,14 +31,17 @@ const ParksSettings: React.FC<{ data: FormData; isLoading: boolean; submit: any
     if (data) {
       setOptions(data.options);
 
-      const { visited } = data;
-      reset({ visited });
+      const { visited, cloudinaryUpload } = data;
+      reset({ visited, cloudinaryUpload });
     }
   }, [data, reset]);
 
   return (
     <Form disabled={formProps.disabled} onSubmit={handleSubmit(submit)}>
       <MultiSelect2 name="visited" options={options} {...formProps} />
+      <div className={classes.textCenter}>
+        <SwitchForm2 {...formProps} name="cloudinaryUpload" label="Upload Images to Cloudinary" />
+      </div>
       <Button {...formProps} type="submit">
         Save
       </Button>
diff --git a/models/app-config.js b/models/app-config.js
index 0d0b00b7..81f5cc7d 100644
--- a/models/app-config.js
+++ b/models/app-config.js
@@ -83,8 +83,6 @@ const appConfigSchema = new mongoose.Schema(
       cloudinaryUpload: { type: Boolean, default: false },
     },
     parks: {
-      active: { type: Boolean, default: false },
-      schedule: { type: String, default: '10 10 0 * * 1' },
       cloudinaryUpload: { type: Boolean, default: false },
       visited: { type: [] },
     },
diff --git a/models/parks-model.js b/models/parks-model.js
index 965f6333..07748430 100644
--- a/models/parks-model.js
+++ b/models/parks-model.js
@@ -1,14 +1,18 @@
 const mongoose = require('mongoose');
 
-module.exports = mongoose.model('parks', {
-  userId: { type: mongoose.Schema.Types.ObjectId, ref: 'user', required: true },
-  parks: [
-    {
-      name: String,
-      imageUrl: String,
-      State: String,
-      location: String,
-    },
-  ],
-  timestamp: { type: Date, default: Date.now, expires: 60 * 60 * 24 * 90 }, // Delete documents after 90 days
-});
+const parksSchema = new mongoose.Schema(
+  {
+    userId: { type: mongoose.Schema.Types.ObjectId, ref: 'user', required: true },
+    parks: [
+      {
+        name: String,
+        imageUrl: String,
+        State: String,
+        location: String,
+      },
+    ],
+  },
+  { timestamps: true }
+);
+
+module.exports = mongoose.model('parks', parksSchema);
diff --git a/nodejs/goodreads.js b/nodejs/goodreads.js
index 0b524762..d8adb77f 100644
--- a/nodejs/goodreads.js
+++ b/nodejs/goodreads.js
@@ -105,6 +105,7 @@ async function uploadImage({ config, data }) {
   try {
     const response = await cloudinaryUploadAsync(data.img, {
       folder: 'books',
+      // TODO: Use userId in public_id
       // Assign a public id so that when we upload an image with the same id, it will replace the previous one
       public_id: `${data.title}-book`
         .replace(/ /g, '-')
diff --git a/nodejs/instagram.js b/nodejs/instagram.js
index 1053bd29..39e8499b 100644
--- a/nodejs/instagram.js
+++ b/nodejs/instagram.js
@@ -78,6 +78,7 @@ async function uploadImages({ config, image }) {
   try {
     const response = await cloudinaryUploadAsync(image.imgUrl, {
       folder: 'instagram',
+      // TODO: Use userId in public_id
       // Assign a public id so that when we upload an image with the same id, it will replace the previous one
       public_id: `${image.id}-instagram`,
       transformation: [{ flags: 'force_strip', height: 160, width: 160, quality: 'auto:good', crop: 'fill' }],
diff --git a/nodejs/music.js b/nodejs/music.js
index 738551c9..40177267 100644
--- a/nodejs/music.js
+++ b/nodejs/music.js
@@ -148,6 +148,7 @@ function getSpotifyArtist(config, artist) {
       try {
         const response = await cloudinaryUploadAsync(data.img, {
           folder: 'music',
+          // TODO: Use userId in public_id
           // Assign a public id so that when we upload an image with the same id, it will replace the previous one
           public_id: `${data.artist}-artist`
             .replace(/ /g, '-')
diff --git a/nodejs/parks.js b/nodejs/parks.js
index 97a683a9..a0357972 100644
--- a/nodejs/parks.js
+++ b/nodejs/parks.js
@@ -24,20 +24,26 @@ exports.get = (userId) =>
 exports.save = (userId) =>
   appConfig
     .get(userId)
-    .then(uploadAllImages)
-    .then((parks) => {
-      const doc = new ParksModel({ parks, userId });
-      return doc.save();
-    });
+    .then((config) => uploadAllImages(config, userId))
+    .then((parks) =>
+      // Keep only 1 questions document in the DB per user. Override the existing doc if it exists
+      ParksModel.findOneAndUpdate(
+        { userId },
+        { parks, userId },
+        {
+          new: true,
+          upsert: true,
+        }
+      ).lean()
+    );
 
-function uploadAllImages(config) {
-  const parkPromises = parksList.map((park) => uploadImage(config, park));
+function uploadAllImages(config, userId) {
+  const parkPromises = parksList.map((park) => uploadImage(config, park, userId));
 
   return Promise.all(parkPromises);
 }
 
-async function uploadImage(config, park) {
-  // NOTE: This config is not availible yet, so we will never upload any images
+async function uploadImage(config, park, userId) {
   if (!config.parks.cloudinaryUpload) {
     return park;
   }
@@ -46,12 +52,17 @@ async function uploadImage(config, park) {
     const response = await cloudinaryUploadAsync(park.imageUrl, {
       folder: 'parks',
       // Assign a public id so that when we upload an image with the same id, it will replace the previous one
-      public_id: `${park.name}-park`
+      public_id: `${userId}-${park.name}-park`
         .replace(/ /g, '-')
         .replace(/[^a-zA-Z0-9-_]/g, '')
         .toLowerCase()
         .substring(0, 100),
       transformation: [
+        {
+          width: 140,
+          height: 140,
+          crop: 'lfill',
+        },
         {
           background: '#786262',
           effect: 'colorize:40',
diff --git a/nodejs/player-fm.js b/nodejs/player-fm.js
index 9b35c077..8ed195ed 100644
--- a/nodejs/player-fm.js
+++ b/nodejs/player-fm.js
@@ -99,6 +99,7 @@ function getArtwork({ config, podcast }) {
       try {
         const response = await cloudinaryUploadAsync(data.imgUrl, {
           folder: 'podcasts',
+          // TODO: Use userId in public_id
           // Assign a public id so that when we upload an image with the same id, it will replace the previous one
           public_id: `${data.title}-podcast`
             .replace(/ /g, '-')