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.

Sign up for GitHub

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

Jump to bottom

[NodeSearchBundle] move mapping to symfony config and allow custom data to be indexed #537

Merged
merged 2 commits into from
Jul 13, 2015
Merged

[NodeSearchBundle] move mapping to symfony config and allow custom data to be indexed #537

merged 2 commits into from
Jul 13, 2015

Conversation

mlebkowski
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Fixed tickets #411

The index mapping configuration is now in bundle config and can be tweaked by the end user easily.

In addition, I have restored the onIndexNode event as well as added an interface to extend the indexed document by adding custom data.

kunstmaan_node_search:
    mapping:
        average_score:
            name: average_score
            type: float
            index: not_analyzed
        tags:
            name: tags
            type: string
            index: analyzed

@roderik roderik added this to the 3.2.1 milestone Jul 13, 2015
roderik pushed a commit that referenced this pull request Jul 13, 2015
Merge pull request #537 from arsthanea/feature/search-configuration
aa2d986 
[NodeSearchBundle] move mapping to symfony config and allow custom data to be indexed
@roderik roderik merged commit aa2d986 into Kunstmaan:master Jul 13, 2015
@mlebkowski mlebkowski deleted the feature/search-configuration branch February 4, 2016 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Priority: Normal Profile: Backend Target audience: Developers Type: Feature
Projects
None yet
Milestone
3.2.1
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mlebkowski @roderik