fix: locator result card address format#1067
Conversation
|
Warning: Component files have been updated but no migrations have been added. See https://github.com/yext/visual-editor/blob/main/packages/visual-editor/src/components/migrations/README.md for more information. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Repository UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
WalkthroughThe LocatorResultCard component was refactored to simplify how the Address component is rendered. Previously, individual address properties (line1, line2, city/region/postalCode) were explicitly constructed and passed to the Address component. The change consolidates this to pass a single address property instead, reducing code complexity and altering the address formatting output. Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
We recently changed all other addresses to use the default pages-components address formatting since it has built-in regionalization support
https://yext.slack.com/archives/C02UVSE7P6W/p1772118800515789