[FEATURE] Include facets from all associated metadata sections for fulltext pages #9
Conversation
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
3 participants
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.
Hallo Chris,
mir ist wohl in den vergangenen Wochen, vllt. auch durch meine Krankheit, ein PR durchgerutscht, den wir auch für die Adressbücher benötigen und der bereits in Kitodo.Presentation im Main-Branch integriert wurde. Das heißt, wir müssen den hier zukünftig auch nicht irgendwie weiter tracken oder so: kitodo#1441
Dabei geht es darum, dass der Indexer alle Facetten von übergeordneten Strukturelementen in die physische Seite schreibt, damit man in Volltextsuchen vollumfänglich facettieren kann. Da hier bereits andere PRs im Main-Branch vorangingen, habe ich deshalb hier einen eigenen PR verfasst, damit wir die nicht irgendwie mitberücksichtigen müssen. In sich ist dieser unabhängig von denen.
Könntest Du bitte den PR mergen und in das Livesystem übertragen? Wenn das passiert ist, bitte einmal eine Rückmeldung an mich, damit die Indexierung nochmal testen kann. Wenn etwas gewaltig schiefgegangen ist (wovon ich eigentlich nicht ausgehe), dann müssten wir den doch nochmal zurückrollen...
Danke und viele Grüße,
Michael