fix: only show users' requests badge if count > 0

RaunoT · Oct 30, 2024 · f021269 · f021269
1 parent 48a17df
commit f021269
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 3 deletions.
diff --git a/.gitignore b/.gitignore
@@ -44,3 +44,4 @@ public/swe-worker-*.js
 
 #config
 config/settings.json
+config/settings-backup.json
diff --git a/src/app/dashboard/users/page.tsx b/src/app/dashboard/users/page.tsx
@@ -64,7 +64,7 @@ async function DashboardUsersContent({ searchParams }: Props) {
   }
 
   const session = await getServerSession(authOptions)
-  const loggedInUserId = session?.user?.id
+  const loggedInUserId = session?.user.id
   const period = getPeriod(searchParams, settings)
   const [usersData, totalDuration, usersCount, totalRequests] =
     await Promise.all([

diff --git a/src/app/settings/layout.tsx b/src/app/settings/layout.tsx
@@ -22,7 +22,7 @@ export default async function SettingsLayout({ children }: Props) {
   const missingSetting = checkRequiredSettings(settings)
 
   if (
-    !session?.user?.isAdmin &&
+    !session?.user.isAdmin &&
     missingSetting !== 'connection.tautulliUrl' &&
     missingSetting !== 'connection.tautulliApiKey'
   ) {

diff --git a/src/components/MediaItem/MediaItem.tsx b/src/components/MediaItem/MediaItem.tsx
@@ -211,7 +211,7 @@ export default function MediaItem({
             </li>
           )}
           {/* Requests */}
-          {activeStats.includes('requests') && isUsers && isOverseerrActive && (
+          {activeStats.includes('requests') && isUsers && data.requests > 0 && (
             <li className='icon-stat-wrapper'>
               <QuestionMarkCircleIcon />
               {pluralize(data.requests, 'request')}
Original file line number	Diff line number	Diff line change
Expand Up		@@ -44,3 +44,4 @@ public/swe-worker-*.js

		#config
		config/settings.json
		config/settings-backup.json