Chatnachrichten in einen neuen Kanal verschoben hat sie alle durcheinandergebracht

JammyDodger · 17. Mai 2022 um 10:33

Ich habe 28 Nachrichten von einem Kanal in einen neuen verschoben und sie sind alle durcheinander geraten:

JammyDodger

#Wie kann Chat Diskussionen anstoßen?

Hmmm. Bin ich der Einzige oder scheinen diese jetzt durcheinander zu sein?

JammyDodger

#Das Chat Feedback Thema

Ich habe mir die durcheinandergeratenen Beiträge in https://meta.discourse.org/chat/channel/147/chat?messageId=2644 noch einmal angesehen, und es scheint, dass sie alle denselben Zeitstempel haben.

martin · 23. Mai 2022 um 01:57

Danke für die Meldung. Ich habe versucht, dies bei der Funktion zu berücksichtigen, aber ich denke, ich hatte mit meinen Tests ein wenig zu viel Glück Wir setzen hier also tatsächlich alle Zeitstempel auf dasselbe:

github.com/discourse/discourse-chat

lib/message_mover.rb

82c0ee7f1


      
          def create_destination_messages_in_channel(destination_channel)
            moved_message_ids = DB.query_single(<<~SQL, message_ids: @ordered_source_message_ids, destination_channel_id: destination_channel.id)
              INSERT INTO chat_messages(chat_channel_id, user_id, message, cooked, cooked_version, created_at, updated_at)
              SELECT :destination_channel_id, user_id, message, cooked, cooked_version, CLOCK_TIMESTAMP(), CLOCK_TIMESTAMP()
              FROM chat_messages
              WHERE id IN (:message_ids)
              RETURNING id
            SQL

Das Problem ist, dass wir die verschobenen Nachrichten nicht zwischen die vorhandenen Kanalnachrichten einfügen wollen, und das wird umso schwieriger, je mehr Nachrichten Sie verschieben.

Eine Frage, bevor ich mich zu sehr damit beschäftige: Können Sie sich erinnern und identifizieren, welche Nachrichten fehl am Platz sind? Sind es nur ein paar oder sind sie komplett durcheinander? Ich glaube, was die Diskrepanz möglicherweise verursacht hat, ist, dass wir beim Abrufen von Nachrichten für einen Kanal diese nach ID-Reihenfolge abrufen (die wir in den meisten Fällen DESC sortieren und dann umkehren):

github.com/discourse/discourse-chat

app/controllers/chat_controller.rb

82c0ee7f1


      
          messages = preloaded_chat_message_query.where(chat_channel: @chat_channel)
          
          order_direction = :desc
          if params[:before_message_id]
            messages = messages.where("id < ?", params[:before_message_id])
          end
          
          if params[:after_message_id]
            messages = messages.where("id > ?", params[:after_message_id])
            order_direction = :asc
          end

github.com/discourse/discourse-chat

app/controllers/chat_controller.rb

82c0ee7f1


      
          chat_view = ChatView.new(
            chat_channel: @chat_channel,
            chat_messages: params[:after_message_id] ? messages : messages.to_a.reverse,

Während ich im Message Mover nach created_at sortiere, um die Reihenfolge beizubehalten, was zu einigen kleinen Diskrepanzen führen könnte:

github.com/discourse/discourse-chat

lib/message_mover.rb

82c0ee7f1


      
          def find_messages(message_ids, channel)
            ChatMessage.where(id: message_ids, chat_channel_id: channel.id).order(:created_at)
          end

Ich habe einige Ideen, wie wir das angehen könnten (vielleicht würde es ausreichen, den Message Mover so zu ändern, dass er nach ID sortiert, oder den Controller so zu ändern, dass er nach created_at sortiert, wobei letzteres bevorzugt wird, da ich denke, dass es sinnvoller wäre), aber ich würde gerne hören, wie durcheinander die Reihenfolge ist, wenn das überhaupt möglich ist.

JammyDodger · 23. Mai 2022 um 06:02

I un-deleted them from the original channel after I noticed they were jumbled in the new one. I should be able to quote them in order here:

Original order

Originally sent in The Chat Feedback topic

JammyDodger

I'm looking for ways we can capture the essence of the new chat flow as a demonstration of how chat can be the seeds of bigger discussion

Has anyone got any ideas on how we can do that from where we are now with the chat-testing here on Meta?

The feedback points are great, and I think they will be calved off into their own proper topics shortly too, but I was hoping for something that would work as a great exemplar for anyone just joining the @chat-testers. Something that people can just look at and go 'Aaaah. Yes, I wasn't sure at first, but I see how can chat can be the pre-cursor to in depth discussion'

I may be asking too much

RGJ

TBH I don't see how chat can be the pre-cursor to in depth discussion

But that might be because I'm old

JammyDodger

So we're looking for an example that will help change @RGJ's mind too

Moin

I like the example here. But that example does not fit here. Maybe the topic could be about a feature you miss just now. Something you wouldn't start a topic because for example you don’t have time to check whether there is a similar feature request or you think no one else will be interested into that.

JammyDodger

@Moin, your search skills are always a blessing

I was looking for one about seeds and trees, but couldn't find it on here

But, yes. Some kind of relaxed/friendly/informal chat in which an idea forms through the course of the easy back-and-forth, and then inspires a proper discussion topic

oshyan

This is 100% my interest and use-case for chat. But can you clarify exactly what you mean by an example of this. Are you interested in, say, samples of a chat (regardless of whether it happened in Discourse chat) that could/should/would have (obviously) led to deeper discussion and/or did lead to deeper discussion but in an arguably less ideal medium for it (chat vs. forum)? If that's the case I may need a little to find some, but I absolutely have good examples from my productivity community. If you're looking for examples in Discourse chat, specifically, that's going to be harder to find. But I absolutely think this is a big value of chat in Discourse, and depending on the community, it will play a bigger or smaller role.

Jonathan_Poyer

I think showcase a discussion around a new feature is actually a nice way to demo it at least during the first moment of that idea. For some people, this spark of discussion is during the time of development or about to begin. There are always more things to discuss about and referring to a topic (or multiple) makes sense.

oshyan

As a perhaps more conceptual example of a situation where chat can (and should) quickly turn into a topic, even mid-way through a discussion, this is something that often happens in software dev management communities I'm part of, as well as my productivity community:

New person joins chat and asks a seemingly simple or innocuous question
Responses from highly informed and/or impassioned regulars quickly grow to 10s of lines of text, paragraph breaks start happening, and that chat channel becomes saturated only with discussion on this one question (topic)
Everything becomes difficult to respond to as each "message" includes lots of points and ideas and no select-to-quote/reply feature makes parsing and responding to each thing difficult
These conversations are also often valuable debates that would quickly disappear in the subsequent flow of a chat, so even retroactively moving them to a Topic is potentially very valuable

JammyDodger

I think I was initially looking for something we could provide as an example topic/chat here on Meta to demonstrate to people new to Discourse Chat of how it could gel nicely with pre-existing ideas of the 'long-form paragraph' view of Discourse

So even something that we created to neatly demonstrate the principle

Though it sounds like you have lots of examples that would make a great discussion topic

I think anything that can help people easily visualise where in their forum structure chat can slot into would be useful. I am open to all ideas

agungor

I feel that this conversation is becoming such an example per se.

Jonathan_Poyer

Feels like something that would become a thread on other platforms would either need to turn into another chat or a split off topic. But at the same time, topic feel also more like long term discussion vs one-off as here?

jimkleiber

Maybe one way is to quote the initial chat message that started the idea into a topic: How can chat seed topic discussions?

This can give visibility to people who aren't in the chat, especially if the topic starts with a question

but ironically I'm answering here lol

Hmm, I just tried to quote my replies here into that same topic but I think I only have the option to quote into a new topic, not an existing one

JammyDodger

I was just mulling that over. I was working out if I could make a topic with each person's chat as a reply somehow, as there were no suitable chat channels to slide them to. But now you've made it I can see that it could have its own chat channel, and we could move this conversation there

jimkleiber

Ah yes, almost like creating the topic that will create the new chat channel

and then the topic can be filled with just quotes from the chat, pulling the highlights

Jumbled

Originally sent in How can chat seed topic discussions?

oshyan

This is 100% my interest and use-case for chat. But can you clarify exactly what you mean by an example of this. Are you interested in, say, samples of a chat (regardless of whether it happened in Discourse chat) that could/should/would have (obviously) led to deeper discussion and/or did lead to deeper discussion but in an arguably less ideal medium for it (chat vs. forum)? If that's the case I may need a little to find some, but I absolutely have good examples from my productivity community. If you're looking for examples in Discourse chat, specifically, that's going to be harder to find. But I absolutely think this is a big value of chat in Discourse, and depending on the community, it will play a bigger or smaller role.

Jonathan_Poyer

I think showcase a discussion around a new feature is actually a nice way to demo it at least during the first moment of that idea. For some people, this spark of discussion is during the time of development or about to begin. There are always more things to discuss about and referring to a topic (or multiple) makes sense.

oshyan

As a perhaps more conceptual example of a situation where chat can (and should) quickly turn into a topic, even mid-way through a discussion, this is something that often happens in software dev management communities I'm part of, as well as my productivity community:

New person joins chat and asks a seemingly simple or innocuous question
Responses from highly informed and/or impassioned regulars quickly grow to 10s of lines of text, paragraph breaks start happening, and that chat channel becomes saturated only with discussion on this one question (topic)
Everything becomes difficult to respond to as each "message" includes lots of points and ideas and no select-to-quote/reply feature makes parsing and responding to each thing difficult
These conversations are also often valuable debates that would quickly disappear in the subsequent flow of a chat, so even retroactively moving them to a Topic is potentially very valuable

JammyDodger

I think I was initially looking for something we could provide as an example topic/chat here on Meta to demonstrate to people new to Discourse Chat of how it could gel nicely with pre-existing ideas of the 'long-form paragraph' view of Discourse

So even something that we created to neatly demonstrate the principle

I think anything that can help people easily visualise where in their forum structure chat can slot into would be useful. I am open to all ideas

I'm looking for ways we can capture the essence of the new chat flow as a demonstration of how chat can be the seeds of bigger discussion

Though it sounds like you have lots of examples that would make a great discussion topic

Has anyone got any ideas on how we can do that from where we are now with the chat-testing here on Meta?

RGJ

TBH I don't see how chat can be the pre-cursor to in depth discussion

agungor

I feel that this conversation is becoming such an example per se.

Jonathan_Poyer

Feels like something that would become a thread on other platforms would either need to turn into another chat or a split off topic. But at the same time, topic feel also more like long term discussion vs one-off as here?

JammyDodger

The feedback points are great, and I think they will be calved off into their own proper topics shortly too, but I was hoping for something that would work as a great exemplar for anyone just joining the @chat-testers. Something that people can just look at and go 'Aaaah. Yes, I wasn't sure at first, but I see how can chat can be the pre-cursor to in depth discussion'

jimkleiber

Maybe one way is to quote the initial chat message that started the idea into a topic: How can chat seed topic discussions?

This can give visibility to people who aren't in the chat, especially if the topic starts with a question

JammyDodger

I think, at least. I'll double-check my thinking before I do anything

I may be asking too much

jimkleiber

but ironically I'm answering here lol

RGJ

But that might be because I'm old

JammyDodger

I was just mulling that over. I was working out if I could make a topic with each person's chat as a reply somehow, as there were no suitable chat channels to slide them to. But now you've made it I can see that it could have its own chat channel, and we could move this conversation there

jimkleiber

Ah yes, almost like creating the topic that will create the new chat channel

JammyDodger

So we're looking for an example that will help change @RGJ's mind too

jimkleiber

and then the topic can be filled with just quotes from the chat, pulling the highlights

Moin

I like the example here. But that example does not fit here. Maybe the topic could be about a feature you miss just now. Something you wouldn't start a topic because for example you don’t have time to check whether there is a similar feature request or you think no one else will be interested into that.

JammyDodger

@Moin, your search skills are always a blessing

I was looking for one about seeds and trees, but couldn't find it on here

But, yes. Some kind of relaxed/friendly/informal chat in which an idea forms through the course of the easy back-and-forth, and then inspires a proper discussion topic

mcwumbly · 23. Mai 2022 um 16:50

Ich frage mich, wann es vorzuziehen ist, Nachrichten zu verschieben, anstatt sie zu zitieren. Vielleicht hängt es davon ab, ob bereits ein Thema existiert oder nicht? Bin mir nicht sicher. In welchen Szenarien wäre es besser, die Leute zu Folgendem zu bewegen?\n\n1. Chat-Nachrichten in einem bestehenden Thema zitieren\n2. Chat-Nachrichten in ein bestehendes Thema verschieben\n3. Chat-Nachrichten in ein neues Thema zitieren\n4. Chat-Nachrichten in ein neues Thema verschieben\n\nDa Chat-Nachrichtenstränge, ähm, gesprächiger sind als Themen, habe ich das Gefühl, dass wir generell das Zitieren eher fördern als das Verschieben wollen.\n\nGibt es Fälle, die Leute beobachtet haben oder im Sinn haben, bei denen Sie denken: „Nein, das Zitieren wäre hier nicht gut. Sie müssen sie definitiv stattdessen verschieben.“?

Moin · 23. Mai 2022 um 16:53

Wenn Sie nur zitieren, kann die Diskussion an zwei Stellen fortgesetzt werden.

mcwumbly · 23. Mai 2022 um 17:14

@Moin schlägst du vor, dass das Verschieben von Nachrichten vorzuziehen wäre, wenn man das wirklich vermeiden möchte?

martin · 23. Mai 2022 um 23:29

Vielen Dank, dass Sie das gemacht haben – das ist völlig durcheinander! Ich muss einige lokale Tests mit größeren Nachrichtensätzen durchführen. Ich denke, zumindest das wird benötigt:

Ich bin jedoch generell beunruhigt über die Sortierung nach ID wegen seltsamer Inkonsistenzen. Ich denke, die Sortierung von Nachrichten nach created_at wäre generell besser für die Kanäle. @j.jaffeux oder @mcwumbly, was denkt ihr darüber? Wenn wir uns dafür entscheiden, dann muss der Nachrichtenverschieber möglicherweise die created_at-Werte künstlich um jeweils etwa 10 ms auseinanderziehen, um eine konsistente Sortierung zu gewährleisten.

Ich denke im Allgemeinen, wenn sie für den aktuellen Kanal völlig irrelevant sind, wäre es besser, sie in einen passenderen Kanal zu verschieben. Wir haben das früher intern oft genutzt, als wir Mattermost verwendeten. Zum Beispiel eine Reihe von Vorfallreaktionen im Kanal general, die zur besseren Aufbewahrung in den Kanal incident verschoben werden sollten. Oder belangloses Geplauder in einem Kanal, das besser in den Kanal random passt.

Ich glaube nicht, dass es in diesen Fällen einen Wert darin gibt, zu zitieren und alten Ballast zurückzulassen, und wie Moin sagt, kann es dann verwirrend werden, wenn die Diskussion an zwei verschiedenen Orten fortgesetzt wird.

Beachten Sie, dass diese beiden Optionen derzeit nicht existieren. Wir haben “In Thema verschieben” entfernt, da es in der anfänglichen Implementierung einen Beitrag pro Chat-Nachricht erstellte und auch die ursprünglichen Nachrichten im Kanal nicht löschte. Wenn wir dies in Zukunft wieder einführen wollen, muss es:

a) Stapel von Nachrichten (z. B. 100 pro Beitrag) mithilfe der Chat-Zitatfunktion zitieren und
b) die Originale im Kanal löschen, um Duplikate zu vermeiden.

mcwumbly · 24. Mai 2022 um 01:14

Ich werde mich enthalten, die Implementierung für die Reihenfolge von Beiträgen zu kommentieren und @j.jaffeux das Wort zu diesem Aspekt überlassen.

Ah, ja. Ich habe nicht gefragt, ob Chatnachrichten innerhalb des Chats verschoben werden sollen, aber ich kann sehen, wie das nützlich sein kann, und es hat nicht das Problem, kurze Texte in lange Texte (oder umgekehrt) „im Beitrag“ umzuwandeln.

Das ergibt Sinn. Ich mag die allgemeine Form des Zitierens eher als „Transkript“, da ich denke, dass es sowieso so gelesen wird. In der Vergangenheit, als ich die Slack-Transkriptfunktion verwendete, habe ich sie oft auch in [details] eingepackt und die Dinge im Hauptbeitrag zusammengefasst.

Ein weiterer Gedanke, den ich in diese Richtung hatte, könnte eine ausgefallenere „Kontext erweitern“-Funktion sein, sodass man eine einzelne Nachricht zitieren, dann aber bei Bedarf weitere Nachrichten inline laden kann, um mehr Kontext aus dem Chat anzuzeigen, ohne das Thema zu verlassen.

Ich bin skeptisch, ob dieser Teil notwendig oder wertvoll ist, wenn man Diskussionen über die Grenze zwischen langsamem und schnellem Fahrspuren hinweg referenziert.

martin · 24. Mai 2022 um 02:02

Dies geschieht nur, wenn Sie „Zum Thema verschieben“ wählen. Warum sollten Sie Dinge im Kanal aufbewahren, wenn Sie sie verschieben wollten? Wir hatten bereits einige interne Diskussionen darüber. Sicher, ein normales Zitieren von Nachrichten in einem Thema würde nichts löschen.

Eine kleine Anekdote für Sie: Die Klasse, die die Zitate generiert, heißt tatsächlich ChatTranscriptService

Das ist interessant, wir haben tatsächlich etwas Ähnliches mit unseren Themenzitaten (das haben Sie wahrscheinlich schon gesehen). Es wäre wahrscheinlich nützlich, etwas mehr Kontext zu erhalten, ohne den Kanal tatsächlich besuchen zu müssen.

sam · 24. Mai 2022 um 03:11

Ich würde sagen, der Anwendungsfall für das Verschieben ist:

Wir haben einen Kanal, der der Diskussion über „Wale“ gewidmet ist.
Eine Gruppe von Leuten beginnt eine intensive Diskussion über „Pinguine“, weil sie vergessen haben, auf „#penguin“ zu klicken, und die Sache heiß wurde.
Ein Moderator greift ein und verschiebt das Pinguin-Gespräch in den Pinguin-Kanal.

Ich denke, das Grundlegende hier ist die Neuordnung.

Ich würde sagen, „fudge created_at“ ist die einzig vernünftige Lösung hier, da man alles in einem Block verschieben möchte? Außerdem wird es technisch gesehen zu dem Zeitpunkt erstellt, an dem es verschoben wird.

mcwumbly · 24. Mai 2022 um 03:11

Ja, ich frage mich, ob es notwendig ist oder ob Zitat/Transkription das ist, worauf man sich konzentrieren sollte, damit es wirklich gut funktioniert.

martin · 24. Mai 2022 um 03:51

Ja, das würde ich zu 100 % tun, wenn unsere normale GET-Nachrichtenroute für einen Kanal nach created_at sortiert wäre. Das möchte ich klären, ich habe mich nur gefragt, ob Joffrey über historisches Wissen dazu hat. Wenn nicht, werde ich beides gleichzeitig ändern.

j.jaffeux · 24. Mai 2022 um 09:32

Ja, ich bin zu 100 % bei Sam und dir Alles auf einmal zu verschieben und ihm die created_at des Zeitpunkts der Verschiebung zu geben, ist meiner Meinung nach der einzig vernünftige Ansatz. Andernfalls öffnet sich eine gigantische Büchse der Pandora… woher weiß ich, wo ich es finden soll? Ungelesene Benachrichtigungen für Dinge erhalten, die vor meinem letzten Lesevorgang erstellt wurden? Nein, nein, nein

martin · 25. Mai 2022 um 00:15

Schön, ich werde das Verschieben-Werkzeug so anpassen, dass die verschobenen Nachrichten in der Zukunft mit einem kleinen, verteilten Inkrement liegen, und die Chat-Nachrichten im allgemeinen Controller nach created_at statt nach ID sortieren

martin · 1. Juni 2022 um 00:31

Ich habe dies gerade zusammengeführt, um das Problem hoffentlich zu beheben:

github.com/discourse/discourse-chat

FIX: Message order consistency

main ← issue/fix-message-ordering-and-message-mover

opened 03:13AM - 30 May 22 UTC

martin-brennan

+27 -15

In https://meta.discourse.org/t/moving-chat-messages-to-a-new-channel-jumbled-th…em-all-up/227301 we were having issues with moved messages between channels being jumbled up. During investigations it was found that the ChatController /messages route ordered messages by ID but everything else, including the message mover, ordered by created_at. This commit makes things consistent by changing the /messages route to also order by created_at, then ID. This should make it so the order is always consistent in the UI and also for moved messages.

Ich habe vorerst nichts unternommen, um created_at künstlich in die Zukunft zu verschieben. Mal sehen, wie das zuerst läuft.

martin · 12. Juni 2022 um 22:00

Dieses Thema wurde nach 11 Tagen automatisch geschlossen. Neue Antworten sind nicht mehr möglich.

Thema		Antworten	Aufrufe
Ability to move chat discussion to a topic? Feature chat	23	1499	16. Mai 2023
Ability to move chat messages Feature chat	4	354	16. Dezember 2023
Will Discourse chat include threads? Feature chat , completed	15	2349	5. Juli 2023
Moving posts into an existing topic doesn't keep chronology Feature completed	44	10052	14. Juni 2023
Introducing chat threads! Announcements chat , new-feature	33	6814	8. Juli 2024

Chatnachrichten in einen neuen Kanal verschoben hat sie alle durcheinandergebracht

Verwandte Themen