> Yeah I think we can just drop the target.name logic and completely
> rely on bufName here. If routeMessage returns [] then it means the
> message should be discarded.

Makes sense!

xse (1):
  Make use of destBuffers when fetching history.

 components/app.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.34.4

From: xse <xse@riseup.net>

Fixes an issue where messages intended to go on the server's buffer end up on their own
---
 components/app.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/components/app.js b/components/app.js
index 0366b85..4aa4194 100644
--- a/components/app.js
+++ b/components/app.js
@@ -1132,7 +1132,7 @@ export default class App extends Component {
 					for (let msg of result.messages) {
 						let destBuffers = this.routeMessage(serverID, msg);
 						for (let bufName of destBuffers) {
-							this.addMessage(serverID, target.name, msg);
+							this.addMessage(serverID, bufName, msg);
 						}
 					}
 				}).catch((err) => {
-- 
2.34.4

gamja/patches/.build.yml: SUCCESS in 57s

[Make use of destBuffers when fetching history][0] v2 from [~xse][1]

[0]: https://lists.sr.ht/~emersion/public-inbox/patches/34841
[1]: xse@riseup.net

✓ #828939 SUCCESS gamja/patches/.build.yml https://builds.sr.ht/~emersion/job/828939

Pushed, thanks!