fix: import missing encodeRecipe in VideoEditor - #1671
Conversation
|
@aaniya22 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. |
👋 Thanks for your PR, @aaniya22!Welcome to Reframe — a browser-based video editor built for everyone 🎬
What happens next
Quick checklist
Useful links
Happy coding! 🎉 |
✅ PR Format Check Passed — @aaniya22Basic 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. |
|
👋 This PR has been inactive for 21 days. It will be automatically closed in 14 days if there's no further activity. If you need help with this PR, please comment or ping a maintainer. |
|
Hi @aaniya22 — thanks for this, and sorry for the wait. I'd like to get this merged, but the ci,type check check is failing. To get it unblocked:
Could you take a look within the next 14 days? If I don't hear back by then I'll close it to keep the queue manageable — but you're very welcome to reopen it any time after that. Thanks again for contributing to Reframe 🙏 |
Description
Fixes a runtime/build error in
VideoEditor.tsxcaused by a missing import.encodeRecipewas being called on line 280 but not imported from@/hooks/useVideoEditor, which broke the build (encodeRecipe is not defined).Added the missing named import. Verified with a clean
npm run build—compiles, lints, type-checks, and all pages generate successfully.
Related Issue
Closes #1660
Type of Contribution
Participant Info
Checklist
bun run lintpasses (no ESLint errors) — confirm withbun run lintbefore checkingbunx tsc --noEmitpasses (no TypeScript errors) — confirm withbunx tsc --noEmitbefore checkingaria-label/ accessible names — not applicable, no new elements addedconsole.logstatements left in