get_new_messages loads all messages each ajax call

I've created a MR for review. It's a bit hard to test since https://www.drupal.org/project/private_message/issues/3424499 🐛 Stuck on undefined URL Needs review is not merged yet.

I found the easiest way to test is to test with two browsers A and B and disable Javascript on B. Login to browser A with user 1 go to your private message chat with user 2. Then, send a message from user 2 on browser B to user 1. Back on browser A you should see the message appear without refreshing.

Also, all messages now no longer load every minute!

Please review!

I'm sort of getting the opposite of this since moving to V3. Now I'm not seeing any new messages via an ajax callback. I need to manually refresh the page to see the new message.

Hi Stephen,

If you disable javascript, you will always need to manually refresh to see new messages appear. Ithink an easier way to test is to include the patch from https://www.drupal.org/project/private_message/issues/3424499 🐛 Stuck on undefined URL Needs review and then test with javascript enabled.

After further testing, this is close to working but there is still an issue. When I send a message to someone, I see it appear in the chat above. However, once the get_new_messages is called, my message that I just sent appears again. It only appears an extra time once, but still not ideal.

Fixed the issue of reloading the message I just sent.

Ready for review!

Tested the MR and it resolves the issue for me. +1