-
Notifications
You must be signed in to change notification settings - Fork 0
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
searhcHits as field in manifestations :) #480
base: master
Are you sure you want to change the base?
Conversation
src/schema/draft/manifestation.js
Outdated
""" | ||
Complex search only !!! - manifestations with search hits | ||
""" | ||
searchHits: [Manifestation] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tror lige så godt vi kan pakke den ind i en SearchResultEntry type med det samme
src/utils/utils.js
Outdated
@@ -366,6 +366,14 @@ export async function resolveWork(args, context) { | |||
profile: context.profile, | |||
}); | |||
|
|||
const resolvedSearchHits = args?.searchHits |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
brug manifestations->all i stedet for at hente data ud igen
} | ||
|
||
type HighLightType { | ||
hit: String |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
HighLightType skal nok justeres, hvis vi når dertil
Vi vil gerne kunne vise hele feltet der er hittet og så markere den del af strengen der er hittet.
Men fint til en start
|
||
type SearchHitsType { | ||
match: Manifestation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
synes match er et godt navn, bravo
No description provided.