Feature/search express entries - #2
Open
broreece wants to merge 10 commits into
Open
Conversation
…f validate candidate
hissy
requested changes
Aug 17, 2026
| (int) $entry->getID() | ||
| ), | ||
| 'title' => $entry->getLabel(), | ||
| 'url' => '', |
Member
There was a problem hiding this comment.
It's better to set URL. Select target page from sitemap per an express entity.
Author
|
@hissy I made the suggested update to use the target page of the express entity. In order to show editors will need to add an express list that points to a detail page: After which the search should correctly use the express entries url. Please let me know if this methodology was wrong here. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Fixes #1
Description
This PR allows adding express entities into the concrete 5 adaptor.
Type of change
How Has This Been Tested?
Checklist:
Caution areas:
Sceenshots:
New config fields for the express entities, NOTE if no handle is inputted instead of all express entities being shown like pages none are shown. If requested I can change this so all express entities are shown in this case.


Search result for the express entity: