Chat 229 typing event #75
Conversation
| @Override | ||
| public void getTypingEvent(Boolean isTyping, String email) { | ||
| log.info("Start getTypingEvent isTyping={}, email={}", isTyping, email); | ||
| contactEventService.updateTypingEvent(email, isTyping); | ||
|
|
||
| Integer topicId = contactEventService.getAllByEmail(email).stream() | ||
| .filter(e -> e.getEventType().equals(EventType.SUBSCRIBED)) | ||
| .findFirst() | ||
| .orElseThrow() | ||
| .getTopicId(); | ||
|
|
||
| chatNotificationService.updateNotificationForAllWhoSubscribedToTopic(topicId); | ||
| } |
There was a problem hiding this comment.
трохи збиває з пантелику назва методу getTypingEvent. метод нічого не повертає хоча в назві get. навпаки він оновлює isTyping і робить оповіщення клієнтів.
There was a problem hiding this comment.
Згоден) Давай виправлю на updateTypingEvent або запропонуй іншу назву.
There was a problem hiding this comment.
updateTypingEvent буде норм
| document.addEventListener('keydown', function() { | ||
| if (!isTyping) { | ||
| isTyping = true; | ||
| sendTyping(isTyping); | ||
| } | ||
|
|
||
| clearTimeout(typingTimer); | ||
| typingTimer = setTimeout(function() { | ||
| isTyping = false; | ||
| sendTyping(isTyping); | ||
| }, 2000); | ||
| }); | ||
|
|
There was a problem hiding this comment.
що буде якщо клієнт буде друкувати та зникне зв'язок? по суті на бекенді залишиться статус що він друкує. або я не знайшов на бекенді де змінюється typingEvent без команди з фронта.
There was a problem hiding this comment.
Не перевіряв цей кейс. Гарне зауваження. Виправлю)
There was a problem hiding this comment.
може це зробити іншою таскою щоб не затримувати фронт?
В гіті його немає бо класс TopicMapperImpl генерується бібліотекою MapStruct автоматично при компіляції. По цьому шляху в проекті можна подиитись його реалізацію Chat/target/classes/com/chat/yourway/mapper |
No description provided.