fix: remove record splash and update sql query in add-splash-entry

src/lib/routes/admin-api/user-splash-controller.ts

@@ -25,34 +25,7 @@ class UserSplashController extends Controller {
         this.logger = config.getLogger('splash-controller.ts');
         this.userSplashService = userSplashService;
 
-        this.post('/', this.recordSplash);
+        this.post('/:id', this.updateSplashSettings);
-        this.put('/:id', this.updateSplashSettings);
-    }
-
-    private async recordSplash(
-        req: IAuthRequest<any, any, ISplashBody, any>,
-        res: Response,
-    ): Promise<void> {
-        const BAD_REQUEST = 400;
-        const { user } = req;
-
-        const { splashId } = req.body;
-
-        if (!splashId) {
-            res.status(BAD_REQUEST).json({
-                error: 'splashId must be present.',
-            });
-            return;
-        }
-
-        const splash = {
-            ...req.body,
-            userId: user.id,
-            seen: req.body.seen || false,
-        };
-
-        const updated = await this.userSplashService.updateSplash(splash);
-        res.json(updated);
     }
 
     private async updateSplashSettings(
@@ -66,9 +39,8 @@ class UserSplashController extends Controller {
             ...req.body,
             splashId: id,
             userId: user.id,
-            seen: req.body.seen || false,
+            seen: true,
         };
-
         const updated = await this.userSplashService.updateSplash(splash);
         res.json(updated);
     }

src/lib/services/user-splash-service.ts

@@ -25,7 +25,7 @@ export default class UserSplashService {
             return [];
         }
         try {
-            const splashs = await (
+            const splashs = (
                 await this.userSplashStore.getAllUserSplashs(user.id)
             ).reduce(
                 (splashObject, splash) => ({

src/migrations/20211109103930-add-splash-entry-for-users.js

@@ -2,8 +2,7 @@ exports.up = function (db, cb) {
     db.runSql(`SELECT * FROM users`, (err, results) => {
         results.rows.forEach((user) => {
             db.runSql(
-                `INSERT INTO user_splash(splash_id, user_id, seen) VALUES (?, ?, ?)`,
+                `INSERT INTO user_splash(splash_id, user_id, seen) SELECT 'environment', u.id, false FROM users u`,
-                ['environments', user.id, false],
             );
         });
         cb();