Skip to content

fix: remove leftover debug console log from useVideoEditor#826

Closed
Pravallika21-nama wants to merge 2 commits into
magic-peach:mainfrom
Pravallika21-nama:clean-fix-console-log
Closed

fix: remove leftover debug console log from useVideoEditor#826
Pravallika21-nama wants to merge 2 commits into
magic-peach:mainfrom
Pravallika21-nama:clean-fix-console-log

Conversation

@Pravallika21-nama
Copy link
Copy Markdown

@Pravallika21-nama Pravallika21-nama commented May 20, 2026

Description

This PR removes leftover debug logging from the useVideoEditor hook.
Closes #812

Changes Made

  • Removed development-only memory logging (console.log)
  • Cleaned up the associated debug useEffect block

Why This Change

The debug logging was not needed and could clutter the console. This improves code cleanliness.

Type of Contribution

  • Bug fix

Notes

No functional changes were made.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 20, 2026

@Pravallika21-nama is attempting to deploy a commit to the magic-peach1's projects Team on Vercel.

A member of the Team first needs to authorize it.

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@github-actions
Copy link
Copy Markdown
Contributor

👋 Thanks for your PR, @Pravallika21-nama!

Welcome to Reframe — a browser-based video editor built for everyone 🎬

What happens next

  1. 🤖 Automated checks — build & TypeScript typecheck will run automatically
  2. Vercel preview — a preview deployment will be created (requires maintainer authorization for fork PRs)
  3. 👀 Code review — a maintainer will review your changes
  4. 🚀 Merge — once approved, your PR will be merged!

Quick checklist

  • PR title follows Conventional Commits (e.g. feat: add dark mode)
  • Linked the issue this PR closes (e.g. Closes #123)
  • Tested the changes locally (bun run dev)
  • Build passes (bun run build)

Useful links

Happy coding! 🎉

@github-actions github-actions Bot added level:advanced Advanced level - 55 pts type:bug Bug fix labels May 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 20, 2026

✅ PR Format Check Passed — @Pravallika21-nama

Basic format checks passed. A maintainer will review your code changes.

This does not mean the PR is approved — it just means the format is correct.

@Pravallika21-nama
Copy link
Copy Markdown
Author

Pravallika21-nama commented May 20, 2026

@HirenGajjar guide me with errors

@HirenGajjar
Copy link
Copy Markdown
Contributor

Hey @Pravallika21-nama,i believe your PR description is missing a linked issue.
Add Closes # to the description and the bot warning should clear up!

@Pravallika21-nama
Copy link
Copy Markdown
Author

@HirenGajjar what changes I need to make you to merge

@HirenGajjar
Copy link
Copy Markdown
Contributor

Hey @Pravallika21-nama, just to clarify — I'm also a contributor here, not a maintainer! I can't merge PRs. I just noticed the missing issue link and thought it might help speed things along for you 🙂

@Pravallika21-nama
Copy link
Copy Markdown
Author

Hi maintainers, thank you for your time and feedback on this PR.

I realized that the changes in this pull request are not aligned properly and may include unintended modifications. To avoid confusion and keep things clean, I’m closing this PR.

I’ll revisit the issue and come back with a smaller, more focused fix if needed. Thanks again!

@Pravallika21-nama Pravallika21-nama deleted the clean-fix-console-log branch May 21, 2026 03:39
@Pravallika21-nama Pravallika21-nama restored the clean-fix-console-log branch May 21, 2026 03:52
@Pravallika21-nama
Copy link
Copy Markdown
Author

Hi maintainers, I checked the current codebase and it looks like there are no remaining debug console logs in useVideoEditor.

Since there’s nothing left to fix, I’m closing this PR to avoid redundancy. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level:advanced Advanced level - 55 pts type:bug Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]Live preview does not update when changing aspect ratio or framing

2 participants