auto load messages (infinite scroll like whatsapp/telegram) and translate messages if LLM configured
adprajap-sjsu2025-11-17 00:39
this patch adds an infinite scroll feature that only loads {batch size} messages at a time. If you configure an LLM for auto translation, we also translate text messages to the selected language. The setting is per user per chat basis, so just because user A wants to translate user B's incoming messages, user B is not forced to do this. Similarly, user A can set user B's messages to be translated to English but all messages from user C to spanish.
adprajap-sjsu2025-11-17 00:40
Hi professor,
I have verified the patch applies on a fresh clone - please let me know if you run into any issues and I will address them before our Tuesday meeting. (if you get the time to look at it, that is!)
Thanks,
Aditya
adprajap-sjsu2025-11-18 10:04
Hi professor,
this patch should apply. I left the office since you were in a meeting - please let me know if you need me to make any changes!
You're still not following Yioop coding conventions for JS variable names which should be snake case not camel case:
+var loadingMessages = false;
+var hasMoreMessages = true;
+var lastScrollTime = 0;
+/*
this patch adds an infinite scroll feature that only loads {batch size} messages at a time. If you configure an LLM for auto translation, we also translate text messages to the selected language. The setting is per user per chat basis, so just because user A wants to translate user B's incoming messages, user B is not forced to do this. Similarly, user A can set user B's messages to be translated to English but all messages from user C to spanish.
Hi professor,
I have verified the patch applies on a fresh clone - please let me know if you run into any issues and I will address them before our Tuesday meeting. (if you get the time to look at it, that is!)
Thanks, Aditya
Hi professor,
this patch should apply. I left the office since you were in a meeting - please let me know if you need me to make any changes!
tl-11-18.patch
Your patch should be adding the files:
updated in this patch
tl-11-23.patch
I meant shouldn't be adding those files
Hi professor,
I don't see those files in the patch? I even removed it from the git ignore
It's part of the patch as a binary file starting line 2466. That's why your patch is so big.
sorry, I was uploading the incorrect file. this patch should be the correct version. Apologies for the inconvenience professor
translation-patch-aditya-11-23.patch
Hey Aditya,
You're still not following Yioop coding conventions for JS variable names which should be snake case not camel case: +var loadingMessages = false; +var hasMoreMessages = true; +var lastScrollTime = 0; +/*
Hi professor, Changed it in this patch
snake-case-variables-11-25.patch
Attachments:
tl-11-16.patch