From 28799bd795c7429593d2788fe47285f14ce33ce3 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 49468b597..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,23 +144,6 @@ public function disable_api(Request $request) return response()->json(['message' => 'API 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 6078cb0dd..3d49f043a 100644 --- a/config/constants.php +++ b/config/constants.php @@ -96,7 +96,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 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 7394d4e16..23317e7e8 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',