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

feat(Edit Mode) #30799 : Creating a REST Endpoint that abstract the Lucene Query generation and return results #31439

Conversation

jcastro-dotcms
Copy link
Contributor

@jcastro-dotcms jcastro-dotcms commented Feb 20, 2025

Proposed Changes

  • Provides a new REST Endpoint that allow users/developers to retrieve content via Lucene from Elasticsearch.
  • Uses the recently created LuceneQueryBuilder service to generate the appropriate query, and passes it down to the API to retrieve contents.
  • The new Endpoint re-uses the part of the code used by the old /api/content/_search that:
    • Retrieves the results from Elasticsearch.
    • Generates the View object with such results and additional search metadata.
  • The re-usable code was moved to the ContentHelper class. All classes in the core project that used such methods were updated, but are NOT part of the goal of this PR.
  • Some code issues were found during the creation of the Postman tests. The respective code fixes are included as well.

This PR fixes: #30799

…ucene Query generation and return results
Copy link

Please use a Conventional Commit title format for this PR. For more information, see https://www.conventionalcommits.org/en/v1.0.0/

@jcastro-dotcms jcastro-dotcms added this pull request to the merge queue Feb 21, 2025
Merged via the queue into main with commit e7686fe Feb 21, 2025
36 checks passed
@jcastro-dotcms jcastro-dotcms deleted the issue-30799-Expose-REST-Endpoint-to-provide-Content-Search-capabilities-PR2 branch February 21, 2025 22:22
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.

[Relationship Field] : Expose REST Endpoint to provide Content Search capabilities
3 participants