Communication: Add send as direct message option for replies

src/main/java/de/tum/cit/aet/artemis/communication/domain/AnswerPost.java

@@ -39,7 +39,7 @@ public class AnswerPost extends Posting {
     private Set<Reaction> reactions = new HashSet<>();
 
     @ManyToOne
-    @JsonIncludeProperties({ "id", "exercise", "lecture", "course", "courseWideContext", "conversation", "author" })
+    @JsonIncludeProperties({ "id", "exercise", "lecture", "course", "courseWideContext", "conversation", "author", "content" })
     private Post post;
 
     @Transient
@@ -50,17 +50,6 @@ public Boolean doesResolvePost() {
         return resolvesPost;
     }
 
-    @Column(name = "has_forwarded_messages")
-    private boolean hasForwardedMessages;
-
-    public boolean getHasForwardedMessages() {
-        return hasForwardedMessages;
-    }
-
-    public void setHasForwardedMessages(boolean hasForwardedMessages) {
-        this.hasForwardedMessages = hasForwardedMessages;
-    }
-
     public void setResolvesPost(Boolean resolvesPost) {
         this.resolvesPost = resolvesPost;
     }

src/main/java/de/tum/cit/aet/artemis/communication/domain/Post.java

@@ -101,8 +101,8 @@ public class Post extends Posting {
     @Transient
     private boolean isSaved = false;
 
-    @Column(name = "has_forwarded_messages")
-    private boolean hasForwardedMessages;
+    @Column(name = "original_answer_id")
+    private Long originalAnswerId;
 
     public Post() {
     }
@@ -119,14 +119,6 @@ public void setTitle(String title) {
         this.title = title;
     }
 
-    public boolean getHasForwardedMessages() {
-        return hasForwardedMessages;
-    }
-
-    public void setHasForwardedMessages(boolean hasForwardedMessages) {
-        this.hasForwardedMessages = hasForwardedMessages;
-    }
-
     public Boolean isVisibleForStudents() {
         return visibleForStudents;
     }
@@ -247,6 +239,14 @@ public void setIsSaved(boolean isSaved) {
         this.isSaved = isSaved;
     }
 
+    public Long getOriginalAnswerId() {
+        return originalAnswerId;
+    }
+
+    public void setOriginalAnswerId(Long originalAnswerId) {
+        this.originalAnswerId = originalAnswerId;
+    }
+
     /**
      * Helper method to extract the course a Post belongs to, which is found in different locations based on the Post's context
      *

src/main/java/de/tum/cit/aet/artemis/communication/domain/Posting.java

@@ -60,6 +60,9 @@ public abstract class Posting extends DomainObject {
     @Transient
     private UserRole authorRoleTransient;
 
+    @Column(name = "has_forwarded_messages")
+    private boolean hasForwardedMessages;
+
     public String getTokenizedContent() {
         return tokenizedContent;
     }
@@ -121,4 +124,12 @@ public void setAuthorRole(UserRole authorRole) {
     public abstract Course getCoursePostingBelongsTo();
 
     public abstract Conversation getConversation();
+
+    public boolean getHasForwardedMessages() {
+        return hasForwardedMessages;
+    }
+
+    public void setHasForwardedMessages(boolean hasForwardedMessages) {
+        this.hasForwardedMessages = hasForwardedMessages;
+    }
 }

src/main/resources/config/liquibase/changelog/20250227143115_changelog.xml

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<databaseChangeLog
+        xmlns="http://www.liquibase.org/xml/ns/dbchangelog"
+        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+        xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog https://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-latest.xsd">
+
+    <changeSet id="20250227143115" author="aykan">
+        <addColumn tableName="post">
+            <column name="original_answer_id" type="BIGINT"/>
+        </addColumn>
+
+        <addForeignKeyConstraint
+                baseTableName="post"
+                baseColumnNames="original_answer_id"
+                constraintName="fk_post_original_answer"
+                referencedTableName="answer_post"
+                referencedColumnNames="id"/>
+    </changeSet>
+
+</databaseChangeLog>

src/main/resources/config/liquibase/master.xml

@@ -56,6 +56,7 @@
     <include file="classpath:config/liquibase/changelog/20250129173003_changelog.xml" relativeToChangelogFile="false"/>
     <include file="classpath:config/liquibase/changelog/20250218181818_changelog.xml" relativeToChangelogFile="false"/>
     <include file="classpath:config/liquibase/changelog/20250222125010_changelog.xml" relativeToChangelogFile="false"/>
+    <include file="classpath:config/liquibase/changelog/20250227143115_changelog.xml" relativeToChangelogFile="false"/>
 
     <!-- NOTE: please use the format "YYYYMMDDhhmmss_changelog.xml", i.e. year month day hour minutes seconds and not something else! -->
     <!-- we should also stay in a chronological order! -->

src/main/webapp/app/entities/metis/post.model.ts

@@ -10,6 +10,8 @@ export class Post extends Posting {
     public plagiarismCase?: PlagiarismCase;
     public displayPriority?: DisplayPriority;
     public resolved?: boolean;
+    public isConsecutive?: boolean = false;
+    public originalAnswerId?: number;
     public forwardedPosts?: Post[] = [];
     public forwardedAnswerPosts?: AnswerPost[] = [];
 

src/main/webapp/app/overview/course-conversations/course-conversations.component.ts

@@ -635,7 +635,15 @@ export class CourseConversationsComponent implements OnInit, OnDestroy {
             this.focusPostId = (post as AnswerPost)?.post?.id;
         }
 
-        this.metisConversationService.setActiveConversation(id);
-        this.changeDetector.detectChanges();
+        if (this.activeConversation && this.activeConversation.id === id) {
+            this.metisConversationService.setActiveConversation(undefined);
+            setTimeout(() => {
+                this.metisConversationService.setActiveConversation(id);
+                this.changeDetector.detectChanges();
+            }, 0);
+        } else {
+            this.metisConversationService.setActiveConversation(id);
+            this.changeDetector.detectChanges();
+        }
     }
 }

src/main/webapp/app/shared/metis/message/message-reply-inline-input/message-reply-inline-input.component.html

@@ -12,7 +12,7 @@
             [isFormGroupValid]="formGroup.valid"
             [editType]="editType"
         />
-        <div class="col mt-1 text-end">
+        <div class="col mt-1">
             @if (!warningDismissed) {
                 <div class="alert alert-warning alert-dismissible text-start fade show" role="alert">
                     <button type="button" class="btn-close" (click)="closeAlert()" aria-label="Close"></button>
@@ -22,23 +22,34 @@ <h4 class="alert-heading" jhiTranslate="artemisApp.messageWarning.headerText"></
                     <p class="mb-0" jhiTranslate="artemisApp.messageWarning.lastParagraph"></p>
                 </div>
             }
-            @if (editType === EditType.UPDATE) {
-                <button
-                    jhi-posting-button
-                    [buttonLabel]="'artemisApp.metis.cancel' | artemisTranslate"
-                    class="btn btn-sm btn-outline-secondary"
-                    (click)="isModalOpen.emit()"
-                ></button>
-                <button
-                    jhi-posting-button
-                    [buttonLoading]="isLoading"
-                    [disabled]="isLoading || !formGroup.valid"
-                    [buttonLabel]="'artemisApp.conversationsLayout.saveMessage' | artemisTranslate"
-                    class="btn btn-sm btn-outline-primary"
-                    id="save"
-                    type="submit"
-                ></button>
-            }
+            <div class="d-flex justify-content-between align-items-center">
+                @if (editType !== EditType.UPDATE) {
+                    <div class="direct-message-checkbox">
+                        <input type="checkbox" id="directMessageCheckbox" [checked]="sendAsDirectMessage()" (change)="toggleSendAsDirectMessage()" />
+                        <label for="directMessageCheckbox" jhiTranslate="artemisApp.conversationsLayout.sendAsDirectMessage"></label>
+                    </div>
+                }
+
+                <div class="d-flex gap-2">
+                    @if (editType === EditType.UPDATE) {
+                        <button
+                            jhi-posting-button
+                            [buttonLabel]="'artemisApp.metis.cancel' | artemisTranslate"
+                            class="btn btn-sm btn-outline-secondary"
+                            (click)="isModalOpen.emit()"
+                        ></button>
+                        <button
+                            jhi-posting-button
+                            [buttonLoading]="isLoading"
+                            [disabled]="isLoading || !formGroup.valid"
+                            [buttonLabel]="'artemisApp.conversationsLayout.saveMessage' | artemisTranslate"
+                            class="btn btn-sm btn-outline-primary"
+                            id="save"
+                            type="submit"
+                        ></button>
+                    }
+                </div>
+            </div>
         </div>
     </div>
 </form>

src/main/webapp/app/shared/metis/message/message-reply-inline-input/message-reply-inline-input.component.ts

@@ -1,4 +1,4 @@
-import { Component, EventEmitter, OnChanges, OnInit, Output, SimpleChanges, ViewEncapsulation, inject, input } from '@angular/core';
+import { Component, EventEmitter, OnChanges, OnInit, Output, SimpleChanges, ViewEncapsulation, inject, input, signal } from '@angular/core';
 import { AnswerPost } from 'app/entities/metis/answer-post.model';
 import { FormsModule, ReactiveFormsModule, Validators } from '@angular/forms';
 import { TranslateDirective } from 'app/shared/language/translate.directive';
@@ -9,6 +9,7 @@ import { PostingMarkdownEditorComponent } from 'app/shared/metis/posting-markdow
 import { ArtemisTranslatePipe } from 'app/shared/pipes/artemis-translate.pipe';
 import { LocalStorageService } from 'ngx-webstorage';
 import { ConversationDTO } from 'app/entities/metis/conversation/conversation.model';
+import { Post } from 'app/entities/metis/post.model';
 
 @Component({
     selector: 'jhi-message-reply-inline-input',
@@ -26,6 +27,8 @@ export class MessageReplyInlineInputComponent extends PostingCreateEditDirective
 
     @Output() valueChange = new EventEmitter<void>();
 
+    sendAsDirectMessage = signal<boolean>(false);
+
     ngOnInit(): void {
         super.ngOnInit();
         this.warningDismissed = !!this.localStorageService.retrieve('chatWarningDismissed');
@@ -45,6 +48,10 @@ export class MessageReplyInlineInputComponent extends PostingCreateEditDirective
         super.ngOnChanges();
     }
 
+    toggleSendAsDirectMessage(): void {
+        this.sendAsDirectMessage.set(!this.sendAsDirectMessage());
+    }
+
     /**
      * resets the answer post content
      */
@@ -65,18 +72,40 @@ export class MessageReplyInlineInputComponent extends PostingCreateEditDirective
      */
     createPosting(): void {
         this.posting.content = this.formGroup.get('content')?.value;
-        this.metisService.createAnswerPost(this.posting).subscribe({
-            next: (answerPost: AnswerPost) => {
-                this.resetFormGroup('');
-                this.isLoading = false;
-                this.onCreate.emit(answerPost);
-            },
-            error: () => {
-                this.isLoading = false;
+        this.isLoading = true;
+
+        const createAnswerPost$ = this.metisService.createAnswerPost(this.posting);
+
+        createAnswerPost$.subscribe({
+            next: (createdAnswerPost: AnswerPost) => {
+                if (this.sendAsDirectMessage()) {
+                    const newPost = this.mapAnswerPostToPost(createdAnswerPost);
+                    this.metisService.createPost(newPost).subscribe({
+                        next: () => this.finalizeCreation(createdAnswerPost),
+                        error: () => (this.isLoading = false),
+                    });
+                } else {
+                    this.finalizeCreation(createdAnswerPost);
+                }
             },
+            error: () => (this.isLoading = false),
         });
     }
 
+    private finalizeCreation(answerPost: AnswerPost): void {
+        this.resetFormGroup('');
+        this.isLoading = false;
+        this.onCreate.emit(answerPost);
+    }
+
+    private mapAnswerPostToPost(answerPost: AnswerPost): Post {
+        return {
+            content: answerPost.content,
+            conversation: this.activeConversation(),
+            originalAnswerId: answerPost.id,
+        } as Post;
+    }
+
     /**
      * invokes the metis service with the updated answer post
      * ends the process successfully by closing the modal and stopping the button's loading animation

src/main/webapp/app/shared/metis/post/post.component.html

@@ -15,7 +15,7 @@
             {{ posting.creationDate | artemisDate: 'time' }}
         </span>
     }
-    @if (!isConsecutive()) {
+    @if (!isConsecutive() || posting.originalAnswerId) {
         <div class="ps-3">
             <jhi-posting-header
                 [previewMode]="previewMode()"
@@ -29,6 +29,18 @@
             />
         </div>
     }
+    @if (posting.originalAnswerId) {
+        <div class="message-container post-content-padding text-muted fs-smaller mt-1">
+            <div class="message-content">
+                <span class="flex-wrapper">
+                    <span class="replied-text" jhiTranslate="artemisApp.metis.post.repliedToThread"></span>
+                    <a (click)="onTriggerNavigateToPost(originalAnswer!)" class="one-line">
+                        {{ originalAnswer?.post?.content }}
+                    </a>
+                </span>
+            </div>
+        </div>
+    }
     <div class="align-items-center">
         <div>
             <div class="post-context-information-wrap post-content-padding" [ngClass]="{ 'is-saved': isConsecutive() && posting.isSaved }">
@@ -194,10 +206,19 @@
                 <span jhiTranslate="artemisApp.metis.post.deleteMessage"></span>
             </button>
         }
-        <button class="dropdown-item d-flex" (click)="isCommunicationPage ? openThread.emit() : reactionsBarComponent()?.openAnswerView()">
-            <fa-icon [icon]="faComments" class="item-icon" />
-            <span jhiTranslate="artemisApp.metis.post.replyMessage"></span>
-        </button>
+        @if (!isThreadSidebar) {
+            @if (!posting.originalAnswerId) {
+                <button class="dropdown-item d-flex" (click)="isCommunicationPage ? openThread.emit() : reactionsBarComponent()?.openAnswerView()">
+                    <fa-icon [icon]="faComments" class="item-icon" />
+                    <span jhiTranslate="artemisApp.metis.post.replyMessage"></span>
+                </button>
+            } @else {
+                <button class="dropdown-item d-flex" (click)="isCommunicationPage ? onTriggerNavigateToPost(originalAnswer!) : reactionsBarComponent()?.openAnswerView()">
+                    <fa-icon [icon]="faComments" class="item-icon" />
+                    <span jhiTranslate="artemisApp.metis.post.viewThread"></span>
+                </button>
+            }
+        }
         <button class="dropdown-item d-flex" (click)="toggleSavePost()">
             <fa-icon [icon]="posting.isSaved ? faBookmark : farBookmark" class="item-icon" />
             <span [jhiTranslate]="posting.isSaved ? 'artemisApp.metis.post.removeBookmarkPost' : 'artemisApp.metis.post.bookmarkPost'"></span>

src/main/webapp/app/shared/metis/post/post.component.scss

@@ -105,3 +105,23 @@
     visibility: visible;
     opacity: 1;
 }
+
+.flex-wrapper {
+    display: flex;
+    align-items: center;
+    width: 100%;
+    gap: 0.3rem;
+}
+
+.replied-text {
+    flex: 0 0 auto;
+    white-space: nowrap;
+}
+
+.one-line {
+    flex: 1 1 0;
+    min-width: 0;
+    white-space: nowrap;
+    overflow: hidden;
+    text-overflow: ellipsis;
+}