-
Notifications
You must be signed in to change notification settings - Fork 0
implemented updatedBy field in concept, and the functionality to sear… #53
Conversation
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.
Ser bra ut, men berre et spørsmål.
@@ -205,6 +216,7 @@ trait ConverterService { | |||
source = concept.source, | |||
created = clock.now(), | |||
updated = clock.now(), | |||
updatedBy = Seq(userInfo.id), |
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.
Riskerer du ikkje å overskrive lista med brukerider her?
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.
I det tilfelle så er det slik at conceptet finnes ikke med denne id'en, mens det finnes en rad med samme id i databasen. I det tilfelle lager vi et helt nytt concept så ingenting blir overskrivet siden det ikke er noe fra før av
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.
Ok. Då skal eg ikkje krangle på det.
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.
LGTM
@@ -56,6 +56,7 @@ trait DraftConceptIndexService { | |||
dateField("lastUpdated"), | |||
keywordField("status.current"), | |||
keywordField("status.other"), | |||
keywordField("users"), |
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.
Syns at users
er et lite beskrivende navn
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.
Kunne vore updatedBy
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.
@N4G1 kan du ikkje lage en ny pr der du kaller det updatedBy? Søket kan fremdeles være users som det er i search-api.
…ch by users
https://github.com/NDLANO/Issues/issues/2107