(cache)(fix): don't send offset and finish if they weren't explicitly provided (!358) · Merge Requests · Minds / Minds Frontend · GitLab

(fix): don't send offset and finish if they weren't explicitly provided

closes #380

Edited 20 minutes ago by Marcelo Rivera

Request to merge eiennohi:sprint/GiddyGiraffe.fix.duplicated-messages-issue-in-messenger into master

The source branch is 2 commits behind the target branch

Open in Web IDE

0/1 discussion resolved

Marcelo Rivera @eiennohi mentioned in merge request engine!211 (closed) 16 hours ago

mentioned in merge request engine!211 (closed)

Mark Harding @markeharding started a discussion on an old version of the diff 5 hours ago

Last updated by Marcelo Rivera 20 minutes ago

src/app/modules/messenger/conversation/conversation.component.ts

169	167	fromSelf = true;
170	168	}
171	169
172	170	this.load({ finish: message.guid });
Mark Harding @markeharding · 5 hours ago Owner @eiennohi why not just pass through `limit: 1` here rather than changing the backend? Marcelo Rivera @eiennohi changed this line in version 2 of the diff 20 minutes ago changed this line in version 2 of the diff Marcelo Rivera @eiennohi · 20 minutes ago Developer done. closing the other MR Please register or sign in to reply

Marcelo Rivera @eiennohi added 1 commit 20 minutes ago
added 1 commit

302dc6c0 - (fix): send limit: 1 when listening from socket

Compare with previous version
Marcelo Rivera @eiennohi changed the description 20 minutes ago

changed the description
You're only seeing other activity in the feed. To add a comment, switch to one of the following options.

Please register or sign in to reply