From d95d2a6c0c1bf294dd4bf2f198580601d460c6b6 Mon Sep 17 00:00:00 2001 From: rosslh Date: Mon, 16 Feb 2026 21:39:11 -0800 Subject: [PATCH] chore: remove broken feedback form and all entrypoints The feedback form relies on instance SMTP being configured, which MapleDeploy customers won't do. Without SMTP it silently logs to the server and shows a misleading success message. Removed the navbar button, onboarding modal, subscription page link, API endpoint, and all associated dead code. --- app/Http/Controllers/Api/OtherController.php | 14 ----- app/Livewire/Help.php | 56 ------------------- config/constants.php | 1 - resources/views/components/navbar.blade.php | 15 ----- resources/views/emails/help.blade.php | 5 -- .../views/livewire/boarding/index.blade.php | 9 --- resources/views/livewire/help.blade.php | 12 ---- .../livewire/subscription/index.blade.php | 3 +- routes/api.php | 2 - 9 files changed, 1 insertion(+), 116 deletions(-) delete mode 100644 app/Livewire/Help.php delete mode 100644 resources/views/emails/help.blade.php delete mode 100644 resources/views/livewire/help.blade.php diff --git a/app/Http/Controllers/Api/OtherController.php b/app/Http/Controllers/Api/OtherController.php index 8f2ba25c8..b87adac5c 100644 --- a/app/Http/Controllers/Api/OtherController.php +++ b/app/Http/Controllers/Api/OtherController.php @@ -4,7 +4,6 @@ use App\Http\Controllers\Controller; use Illuminate\Http\Request; -use Illuminate\Support\Facades\Http; use OpenApi\Attributes as OA; class OtherController extends Controller @@ -145,19 +144,6 @@ public function disable_api(Request $request) return response()->json(['message' => 'API disabled.'], 200); } - public function feedback(Request $request) - { - $content = $request->input('content'); - $webhook_url = config('constants.webhooks.feedback_discord_webhook'); - if ($webhook_url) { - Http::post($webhook_url, [ - 'content' => $content, - ]); - } - - return response()->json(['message' => 'Feedback sent.'], 200); - } - #[OA\Get( summary: 'Healthcheck', description: 'Healthcheck endpoint.', diff --git a/app/Livewire/Help.php b/app/Livewire/Help.php deleted file mode 100644 index 29bd05aa2..000000000 --- a/app/Livewire/Help.php +++ /dev/null @@ -1,56 +0,0 @@ -validate(); - $this->rateLimit(3, 30); - - $settings = instanceSettings(); - $mail = new MailMessage; - $mail->view( - 'emails.help', - [ - 'description' => $this->description, - ] - ); - $mail->subject("[HELP]: {$this->subject}"); - $type = set_transanctional_email_settings($settings); - - // MapleDeploy branding: feedback sent to MapleDeploy support - if (blank($type)) { - // No external API — log locally when SMTP not configured - \Illuminate\Support\Facades\Log::info('Feedback from '.auth()->user()?->email.': '.$this->subject.' — '.$this->description); - } else { - send_user_an_email($mail, auth()->user()?->email, 'support@mapledeploy.ca'); - } - $this->dispatch('success', 'Feedback sent.', 'We will get in touch with you as soon as possible.'); - $this->reset('description', 'subject'); - } catch (\Throwable $e) { - return handleError($e, $this); - } - } - - public function render() - { - return view('livewire.help')->layout('layouts.app'); - } -} diff --git a/config/constants.php b/config/constants.php index 3f46462ff..bcfdf785e 100644 --- a/config/constants.php +++ b/config/constants.php @@ -92,7 +92,6 @@ ], 'webhooks' => [ - 'feedback_discord_webhook' => null, 'dev_webhook' => env('SERVEO_URL'), ], diff --git a/resources/views/components/navbar.blade.php b/resources/views/components/navbar.blade.php index 0dbdfef93..b3af1e91e 100644 --- a/resources/views/components/navbar.blade.php +++ b/resources/views/components/navbar.blade.php @@ -348,21 +348,6 @@ class="{{ request()->is('onboarding*') ? 'menu-item-active menu-item' : 'menu-it @endif -
  • - - -
    - - - - Feedback -
    -     - -     -
  • @csrf diff --git a/resources/views/emails/help.blade.php b/resources/views/emails/help.blade.php deleted file mode 100644 index 6e7abba2c..000000000 --- a/resources/views/emails/help.blade.php +++ /dev/null @@ -1,5 +0,0 @@ -{{ $description }} - -{{ Illuminate\Mail\Markdown::parse('---') }} - -{{-- {{ Illuminate\Mail\Markdown::parse($debug) }} --}} diff --git a/resources/views/livewire/boarding/index.blade.php b/resources/views/livewire/boarding/index.blade.php index 25af14a56..7fd48d05e 100644 --- a/resources/views/livewire/boarding/index.blade.php +++ b/resources/views/livewire/boarding/index.blade.php @@ -706,15 +706,6 @@ class="dark:text-neutral-400 hover:text-coollabs dark:hover:text-warning hover:u Restart - - - - - - @endif \ No newline at end of file diff --git a/resources/views/livewire/help.blade.php b/resources/views/livewire/help.blade.php deleted file mode 100644 index 1dd2d6a5b..000000000 --- a/resources/views/livewire/help.blade.php +++ /dev/null @@ -1,12 +0,0 @@ -
    - {{-- MapleDeploy branding --}} -
    Your feedback helps us improve MapleDeploy. Thank you!
    - - - -
    - Send - -
    diff --git a/resources/views/livewire/subscription/index.blade.php b/resources/views/livewire/subscription/index.blade.php index 70e293698..ff81571d2 100644 --- a/resources/views/livewire/subscription/index.blade.php +++ b/resources/views/livewire/subscription/index.blade.php @@ -57,8 +57,7 @@

    Subscription

    - You are not an admin so you cannot manage your Team's subscription. If this does not make sense, please - contact us. + You are not an admin so you cannot manage your team's subscription. Please contact your team admin. @endif diff --git a/routes/api.php b/routes/api.php index 8ff1fd1cc..fa984ed09 100644 --- a/routes/api.php +++ b/routes/api.php @@ -25,8 +25,6 @@ Route::get('/health', [OtherController::class, 'healthcheck']); }); -Route::post('/feedback', [OtherController::class, 'feedback']); - Route::group([ 'middleware' => ['auth:sanctum', 'api.ability:write'], 'prefix' => 'v1',