Facets integration

Created on 8 December 2020, about 4 years ago
Updated 23 June 2024, 6 months ago

It would be nice if this module could integrate with Facets module to provide "Transform entity ID to label" option.

Currently, facet provides only "DE", "US" ... country name labels. This is not so convenient for search purposes. "Germany", "United States" ... would be much better.

✨ Feature request
Status

Fixed

Version

1.0

Component

Code

Created by

πŸ‡­πŸ‡·Croatia devad

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡«πŸ‡·France fmb PerpinyΓ , Catalonia, EU

    In my opinion, the correct way would be to write a Facets plugin.

  • @fmb opened merge request.
  • Status changed to Needs review over 1 year ago
  • πŸ‡«πŸ‡·France fmb PerpinyΓ , Catalonia, EU

    In order to test this MR, check "Country name" in facets settings.

  • πŸ‡³πŸ‡±Netherlands batigolix Utrecht

    I tested this and it works nice.

    What is missing is documentation that explains how to set it up (especially the "Country name" setting in the facet.)
    But as this module does not have a README, we would need a separate issue for this.

    So I mark this RTBC.

  • Status changed to RTBC over 1 year ago
  • πŸ‡³πŸ‡±Netherlands batigolix Utrecht
  • πŸ‡¬πŸ‡§United Kingdom dakala Fairford, Glos

    @batigolix Thanks for all your work on issues. Would you be able to help update the README please? Once that's done, we can merge and release a new version. Once again, thanks.

  • Status changed to Needs work over 1 year ago
  • πŸ‡³πŸ‡±Netherlands batigolix Utrecht
  • πŸ‡³πŸ‡±Netherlands batigolix Utrecht

    @dakala: the fork needs to be updated so that it contains a readme so it becomes possible to update it. But I dont know how to update the fork.

    As an alternative I can create a new ticket for updating the readme.

  • Status changed to Fixed over 1 year ago
  • πŸ‡¬πŸ‡§United Kingdom dakala Fairford, Glos

    @batigolix: No worries. I don't know how to update forks either. Let's go for the easiest option. I'll merge this now, since it's been tested and marked RTBC before. I'll leave the update to the README in your able hands.

  • πŸ‡­πŸ‡·Croatia devad

    Thanks all. I tried a .dev committed Facets plugin and it works nicely.

  • Automatically closed - issue fixed for 2 weeks with no activity.

  • Status changed to Fixed about 1 year ago
  • πŸ‡³πŸ‡±Netherlands batigolix Utrecht

    @dakala:

    I created an issue (with patch) for the improvement of the readme: ✨ Improve README.md Needs review

  • πŸ‡΅πŸ‡ΉPortugal introfini

    To assist others facing the same issue, ensure that the "Ignore Case" processor is not enabled on your index for the country field. The country codes are stored in uppercase, and the countryManager->getList() method returns keys in uppercase as well. Any conversion to lowercase will cause the functionality to break.

Production build 0.71.5 2024