Problem/Motivation

The README says:

> No other modules are required; we're supporting Drupal Core's search as a source for creating facets.

but the project page says that Facets 3 no longer supports core search:

> Facets 3.x only supports Search API, and introduces "Facets exposed filters" next to "Facet blocks".

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork facets-3506177

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

joachim created an issue. See original summary.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented

You're right, we should update this, only in the 3.x version of the readme.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
Issue tags: +Novice

Tagging as novice

juandhr’s picture

juandhr commented
Assigned: Unassigned » juandhr

Working on it.

juandhr opened merge request !295

juandhr’s picture

juandhr commented
Assigned: juandhr » Unassigned
Status: Active » Needs review

Proposed lines in README.md:

Search API is required: Facets only supports Search API as a source for creating facets. Drupal Core's search is no longer supported as a source for facets.

danrod’s picture

danrod
Location Ottawa
commented

These lines look for me too, so RTBC+1

kieran.cott’s picture

kieran.cott
Location 🇵🇭 Manila
commented

MR 295 makes sense to me as well, RTBC++

joachim’s picture

joachim commented

> Search API is required: Facets only supports Search API as a source for creating facets. Drupal Core's search is no longer supported as a source for facets.

Documentation should not refer to things that don't exist any more.

kieran.cott’s picture

kieran.cott
Location 🇵🇭 Manila
commented

Rephrased to:

Search API is required. Facets supports Search API displays as facet sources.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
Status: Needs review » Reviewed & tested by the community

Looks good.

  • strykaizer committed 3a5f385b on 3.0.x authored by juandhr 
    Issue #3506177 by juandhr: Modify support mentions for Drupal core's...
strykaizer’s picture

strykaizer
Primary language Dutch
Location Belgium
commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.