implement feedback

Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>

Author: lukasdotcom

lib/Controller/LogController.php

@@ -9,8 +9,11 @@
 
 use OCA\ContextChat\Logger;
 use OCP\AppFramework\Controller;
-use OCP\AppFramework\Http\Attribute\NoCSRFRequired;
+use OCP\AppFramework\Http;
 
+use OCP\AppFramework\Http\Attribute\NoCSRFRequired;
+use OCP\AppFramework\Http\Response;
+use OCP\AppFramework\Http\TextPlainResponse;
 use OCP\AppFramework\Http\ZipResponse;
 use OCP\IRequest;
 
@@ -25,10 +28,10 @@ public function __construct(
 	/**
 	 * Downloads log file
 	 *
-	 * @return ZipResponse
+	 * @return Response
 	 */
 	#[NoCSRFRequired]
-	public function getNextcloudLogs(): ZipResponse {
+	public function getNextcloudLogs(): Response {
 		$logFilepath = $this->logger->getLogFilepath();
 		$response = new ZipResponse($this->request, 'nextcloud-logs');
 		$filePath = $logFilepath;
@@ -39,6 +42,9 @@ public function getNextcloudLogs(): ZipResponse {
 			$response->addResource($remoteFile, basename($filePath), $size, $time);
 			$filePath = $logFilepath . '.' . $counter++;
 		}
+		if ($counter == 1) {
+			return new TextPlainResponse('No log file found', Http::STATUS_NOT_FOUND);
+		}
 		return $response;
 	}
 }

src/components/ViewAdmin.vue

@@ -43,16 +43,20 @@ SPDX-License-Identifier: AGPL-3.0-or-later
 					</td>
 				</tr>
 			</table>
-			<h2>{{ t('context_chat', 'Context Chat Logs') }}</h2>
-			<NcButton :href="downloadURLNextcloudLogs">
-				{{ t('context_chat', 'Download Nextcloud logs') }}
-			</NcButton>
-			<NcButton :href="downloadURLDockerLogs">
-				{{ t('context_chat', 'Download Backend logs') }}
-			</NcButton>
 			<p>&nbsp;</p>
 			<p>{{ t('context_chat', 'Eligible files for indexing: {count}', {count: stats.eligible_files_count}) }}</p>
 			<p>{{ t('context_chat', 'Queued content update actions: {count}', {count: stats.queued_actions_count}) }}</p>
+		</NcSettingsSection>
+		<NcSettingsSection :name="t('context_chat', 'Logs')">
+			<div class="horizontal-flex">
+				<NcButton :href="downloadURLNextcloudLogs">
+					{{ t('context_chat', 'Download the PHP App logs') }}
+				</NcButton>
+				<NcButton :href="downloadURLDockerLogs">
+					{{ t('context_chat', 'Download the Ex-App Backend logs') }}
+				</NcButton>
+			</div>
+			<p>&nbsp;</p>
 			<p>
 				<a href="https://docs.nextcloud.com/server/latest/admin_manual/ai/app_context_chat.html">{{
 					t('context_chat', 'Official documentation')
@@ -143,4 +147,9 @@ figure[class^='icon-'] {
 #context_chat a:link, #context_chat a:visited, #context_chat a:hover {
 	text-decoration: underline;
 }
+
+.horizontal-flex {
+	display: flex;
+	gap: 10px;
+}
 </style>