fix: remove duplicated JSX breaking the Dashboard build#333
Conversation
The Dashboard merge (weDevsOfficial#327) left a duplicated closing </a> and a stray ArrowTopRightOnSquareIcon in the Recently Updated table, so `npm run build` failed with a Babel syntax error: ERROR in ./src/components/Dashboard/index.js Module build failed ... Unexpected token (251:26) develop could not produce a build, which blocks the wp.org deploy workflow. Removing the two stray lines restores a clean compile.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
WalkthroughThe Dashboard’s “Recently Updated” table now renders the “View” link with corrected icon markup. No state, data-fetching, or component-structure changes are included. ChangesDashboard Link Markup
Estimated code review effort: 1 (Trivial) | ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Problem
developdoes not build. The Dashboard merge (#327) left a duplicated closing</a>plus a strayArrowTopRightOnSquareIconin the Recently Updated table:Since
assets/build/is CI-generated and the 10up deploy ships the built working tree, a failingnpm run buildblocks the wp.org release entirely.Fix
Remove the two stray lines in
src/components/Dashboard/index.js.Verification
npm run build— webpack compiles clean (warnings only, no errors).Blocks the 2.4.0 release.
Summary by CodeRabbit