Merge pull request #402 from codeigniter4projects/limit-news

fix: Limit news posts grabbed from the forums and fix deployment.
codeigniter4projects · Sep 29, 2023 · 0410ccc · 0410ccc
2 parents bcbd7d6 + 89de25c
commit 0410ccc
Show file tree

Hide file tree

Showing 3 changed files with 316 additions and 342 deletions.
diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
@@ -1,12 +1,12 @@
-name: 'Deploy to Production'
+name: "Deploy to Production"
 
 on:
   push:
-    branches: [ master ]
+    branches: [master]
 
 jobs:
   deploy:
-    name: 'Deploy to Production'
+    name: "Deploy to Production"
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
@@ -68,4 +68,4 @@ jobs:
             echo $'Set up Links\n'
             cd $RELEASE_DIR
             sudo ln -nsf $RELEASE_DIR/$RELEASE "../current"
-            sudo service php7.4-fpm reload
+            sudo service php8.1-fpm reload
diff --git a/app/Models/MyBBModel.php b/app/Models/MyBBModel.php
@@ -66,6 +66,7 @@ public function getRecentNews(int $limit = 5, string $order = 'desc')
         $query   = $builder->select('tid, subject, username, lastpost, lastposter')
             ->where($where)
             ->whereIn('username', $admins)
+            ->limit($limit, 0)
             ->orderBy('lastpost', $order)
             ->get();