ajouts

apps/datahub/src/app/record/record-user-feedbacks/record-user-feedbacks.component.ts

@@ -90,11 +90,11 @@ export class RecordUserFeedbacksComponent implements OnInit, OnDestroy {
 
   onNewAnswer(newUserFeedback: UserFeedback) {
     this.userFeedBackService.postUserFeedback(newUserFeedback)
-    // this.cdr.markForCheck()
   }
 
   publishNewComment() {
-    const newFeedback = {
+    const newFeedback: UserFeedback = {
+      uuid: undefined,
       comment: this.newComment,
       parentUuid: null,
       date: undefined,
@@ -105,6 +105,5 @@ export class RecordUserFeedbacksComponent implements OnInit, OnDestroy {
 
     this.userFeedBackService.postUserFeedback(newFeedback)
     this.newComment = ''
-    this.cdr.markForCheck()
   }
 }

libs/api/repository/src/lib/gn4/platform/gn4-platform.mapper.ts

@@ -70,7 +70,7 @@ export class Gn4PlatformMapper {
     })
   }
 
-   userFeedbacksFromApi(userFeedbacks: any[]): UserFeedback[] { // TODO <--------------------------------------------------------------------- TODO: TYPER
+   userFeedbacksFromApi(userFeedbacks: any[]): UserFeedback[] {
     return userFeedbacks.map(userFeedback => ({
       uuid: userFeedback.uuid,
       comment: userFeedback.comment,

libs/common/domain/src/lib/model/record/user-feedbacks.model.ts

@@ -1,10 +1,10 @@
 export interface UserFeedback {
-  uuid?: string
-  comment?: string
-  authorUserId?: number
-  authorName?: string
-  authorEmail?: string
-  published?: boolean
+  uuid: string
+  comment: string
+  authorUserId: number
+  authorName: string
+  authorEmail: string
+  published: boolean
   parentUuid?: string
-  date?: string
+  date: string
 }

libs/ui/elements/src/lib/user-feedback-item/user-feedback-item.component.ts

@@ -48,9 +48,10 @@ export class UserFeedbackItemComponent implements OnInit {
   publishNewAnswer() {
     this.newAnswerEmitter.emit({
       ...this.userFeedback,
+      uuid: undefined,
       comment: this.newAnswer,
       parentUuid: this.userFeedback.uuid,
-      authorUserId: Number.parseInt(this.activeUser?.id), // todo: il faut faire quelque chose avec ces userId number alors que tout le reste est en uuid...
+      authorUserId: Number.parseInt(this.activeUser?.id), // todo: Shouldn't we do something with this? Convert it to uuid like it's done everywhere else ?...
       authorEmail: this.activeUser?.email,
       authorName: `${this.activeUser?.name} ${this.activeUser?.surname}`
     })