[PR #188] [MERGED] fix(ui): remove frontend debug console logs #3444

New Issue

GiteaMirror · 2026-06-18T17:31:20-05:00

GiteaMirror commented

2026-06-18 17:31:20 -05:00

📋 Pull Request Information

Original PR: https://github.com/reconurge/flowsint/pull/188
Author: @Ghraven
Created: 6/11/2026
Status: ✅ Merged
Merged: 6/11/2026
Merged by: @dextmorgn

Base: main ← Head: fix/remove-frontend-debug-logs

📝 Commits (1)

8de2779 fix(ui): remove frontend debug console logs

📊 Changes

3 files changed (+1 additions, -4 deletions)

View changed files

📝 flowsint-app/src/components/dashboard/investigation/sketches-section.tsx (+1 -1)
📝 flowsint-app/src/components/flows/raw-material.tsx (+0 -2)
📝 flowsint-app/src/components/templates/template-editor.tsx (+0 -1)

📄 Description

What changed

Removed leftover frontend console.log calls from raw material loading and template testing flows.
Stopped passing a placeholder click handler that only logged Create sketch; the button is already wrapped by NewSketch.

Problem

A few shipped UI paths still wrote raw data or placeholder messages to the browser console. That can make local debugging noisier and may expose unnecessary API response details during normal app usage.

Before / after

Before:

Opening raw materials logged the fetched materials object.
Running a template test logged the full response object.
Clicking the empty sketches action emitted a placeholder console message before the real NewSketch wrapper handled creation.

After:

These UI paths no longer emit debug-only console output.
Sketch creation behavior remains delegated to the existing NewSketch wrapper.

Verification

git diff --check passes.
corepack yarn install --frozen-lockfile completed successfully.
corepack yarn typecheck is currently blocked by existing project-wide TypeScript errors unrelated to these files, including missing Tiptap modules and table/router typing errors.
A targeted ESLint run is currently blocked because .eslintrc.cjs requires @typescript-eslint/parser, but that parser is not installed by the lockfile.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/reconurge/flowsint/pull/188 **Author:** [@Ghraven](https://github.com/Ghraven) **Created:** 6/11/2026 **Status:** ✅ Merged **Merged:** 6/11/2026 **Merged by:** [@dextmorgn](https://github.com/dextmorgn) **Base:** `main` ← **Head:** `fix/remove-frontend-debug-logs` --- ### 📝 Commits (1) - [`8de2779`](https://github.com/reconurge/flowsint/commit/8de27796b09ffe2753133c5e94cb767d897f08c9) fix(ui): remove frontend debug console logs ### 📊 Changes **3 files changed** (+1 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `flowsint-app/src/components/dashboard/investigation/sketches-section.tsx` (+1 -1) 📝 `flowsint-app/src/components/flows/raw-material.tsx` (+0 -2) 📝 `flowsint-app/src/components/templates/template-editor.tsx` (+0 -1) </details> ### 📄 Description ## What changed - Removed leftover frontend `console.log` calls from raw material loading and template testing flows. - Stopped passing a placeholder click handler that only logged `Create sketch`; the button is already wrapped by `NewSketch`. ## Problem A few shipped UI paths still wrote raw data or placeholder messages to the browser console. That can make local debugging noisier and may expose unnecessary API response details during normal app usage. ## Before / after Before: - Opening raw materials logged the fetched materials object. - Running a template test logged the full response object. - Clicking the empty sketches action emitted a placeholder console message before the real `NewSketch` wrapper handled creation. After: - These UI paths no longer emit debug-only console output. - Sketch creation behavior remains delegated to the existing `NewSketch` wrapper. ## Verification - `git diff --check` passes. - `corepack yarn install --frozen-lockfile` completed successfully. - `corepack yarn typecheck` is currently blocked by existing project-wide TypeScript errors unrelated to these files, including missing Tiptap modules and table/router typing errors. - A targeted ESLint run is currently blocked because `.eslintrc.cjs` requires `@typescript-eslint/parser`, but that parser is not installed by the lockfile. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2026-06-18 17:31:20 -05:00

GiteaMirror closed this issue

2026-06-18 17:31:22 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/flowsint#3444