fix: handle empty package description in table row display#1763
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe pull request modifies the Description column rendering logic in the TableRow component. Previously, a fallback hyphen was passed through the decodeHtmlEntities function alongside the description value. The change reorders this logic so the description (or empty string) is processed first through decoding and HTML tag stripping, with the fallback hyphen only applied if the resulting processed value is empty. This prevents unnecessary processing of the fallback character. Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
🔗 Linked issue
Extra PR for the issue #1681
🧭 Context
In the first PR, we are not handling the scenario where the description contains only HTML tags. We would end up with empty boxes for the description package on the search page. Instead, we want to display dashes (-).