From 97eb0540e728a929bbf2fd9529ef7535640cc7bd 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 | 18 ------ 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 | 3 - 9 files changed, 1 insertion(+), 121 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 f17a4e46b..8a8c9f07e 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 @@ -265,23 +264,6 @@ public function disable_mcp(Request $request) return response()->json(['message' => 'MCP server disabled.'], 200); } - public function feedback(Request $request) - { - $data = $request->validate([ - 'content' => ['required', 'string', 'min:10', 'max:2000'], - ]); - - $webhook_url = config('constants.webhooks.feedback_discord_webhook'); - if ($webhook_url) { - Http::timeout(5)->post($webhook_url, [ - 'content' => $data['content'], - 'allowed_mentions' => ['parse' => []], - ]); - } - - 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 d19c58d9f..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 6f4454ac4..d11cca184 100644 --- a/config/constants.php +++ b/config/constants.php @@ -97,7 +97,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 96ace8a84..81f77c970 100644 --- a/resources/views/components/navbar.blade.php +++ b/resources/views/components/navbar.blade.php @@ -375,21 +375,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 abad70440..847b537c8 100644 --- a/resources/views/livewire/subscription/index.blade.php +++ b/resources/views/livewire/subscription/index.blade.php @@ -55,8 +55,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 cc380b2be..a499b934b 100644 --- a/routes/api.php +++ b/routes/api.php @@ -26,9 +26,6 @@ Route::get('/health', [OtherController::class, 'healthcheck']); }); -Route::post('/feedback', [OtherController::class, 'feedback']) - ->middleware('throttle:feedback'); - Route::group([ 'middleware' => ['auth:sanctum', 'api.ability:write'], 'prefix' => 'v1',