Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate LocationBias component #338

Merged
merged 2 commits into from
Nov 17, 2022
Merged

Conversation

yen-tt
Copy link
Contributor

@yen-tt yen-tt commented Nov 16, 2022

this pr mark LocationBias as deprecated in favor of the new Geolocation component.

I checked if storybook have some feature to inform user that a component is deprecated, there's an open feature request: storybookjs/storybook#9721 for displaying @deprecated props but nothing official yet. I did come across a custom addon: https://storybook.js.org/addons/@etchteam/storybook-addon-status which seems to work quite nicely.

Show "Deprecated" status for location bias stories:
Screen Shot 2022-11-16 at 4 31 20 PM

J=SLAP-2448
TEST=none

@yen-tt yen-tt requested a review from a team as a code owner November 16, 2022 21:34
@coveralls
Copy link

Coverage Status

Coverage remained the same at 84.395% when pulling c042a44 on dev/deprecate-locationbias into a2e8319 on develop.

@yen-tt yen-tt merged commit 3748b15 into develop Nov 17, 2022
@yen-tt yen-tt deleted the dev/deprecate-locationbias branch November 17, 2022 14:44
@cea2aj cea2aj mentioned this pull request May 9, 2023
@cea2aj cea2aj mentioned this pull request Jun 29, 2023
cea2aj added a commit that referenced this pull request Jun 29, 2023
# Version 1.3.0

### Features
- Introduce new Facet components including the new Facets (#360), StandardFacet (#370), NumericalFacet (#375), and HierarchicalFacet (#373) components which support more customizability than the previous Facet components. 
- Support the new RichTextV2 (Lexical) and Markdown fields which have been converted to HTML as FeaturedSnippetDirectAnswers (#379)
- Remove the Yext logo from the search bar (#376)
- Deprecate LocationBias (#338) component in favor of the new Geolocation component (#336)

### Bug fixes
- Improve the UX of the search bar when long autocomplete text is returned which includes the truncation of long text (#378)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants