Skip to content

fix(ui): remove frontend debug console logs#188

Merged
dextmorgn merged 1 commit into
reconurge:mainfrom
Ghraven:fix/remove-frontend-debug-logs
Jun 11, 2026
Merged

fix(ui): remove frontend debug console logs#188
dextmorgn merged 1 commit into
reconurge:mainfrom
Ghraven:fix/remove-frontend-debug-logs

Conversation

@Ghraven

@Ghraven Ghraven commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

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.

@dextmorgn dextmorgn left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks !

@dextmorgn dextmorgn merged commit 3885f8b into reconurge:main Jun 11, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants