Don
10 ديسمبر 2022، 12:44م
1
Hello,
I noticed 2 issues with chat retention days.
1. There are separate settings to set up chat channel retention days
and chat dm retention days
. But it seems it use the chat channel retention days
value on personal message settings page too.
I setted up 2 different retention day.
The chat retention reminders in messages show the correct value.
2. When the chat channel retention days
and/or chat dm retention days
set to 0
it means retain messages forever. But now on (channel / personal) retention day messages it seems like it saved for 0
day.
Thank you
3 إعجابات
piffy
9 فبراير 2023، 5:36ص
3
Noticed this today where chat history is indefinite
Chat history will be saved for 0 days.
Maybe for 0, just hide that text entirely?
j.jaffeux
(Joffrey Jaffeux)
14 فبراير 2023، 10:56م
4
This is all valid reports, thanks
I have fixed this in a local branch, but I need to check something before I push it hopefully tomorrow.
3 إعجابات
j.jaffeux
(Joffrey Jaffeux)
15 فبراير 2023، 1:50م
5
That should fix it, let me know how it goes for you:
discourse:main
← jjaffeux:chat/reminder
opened 12:02PM - 15 Feb 23 UTC
- Moves logic into one specialized component
- Adds more tests
- Removes dupli… cate key
- Uses pluralization
- Handles 0 case properly
Thanks for the report @Don , good catch
3 إعجابات
Don
15 فبراير 2023، 5:09م
6
Hello @j.jaffeux ,
Thanks for the fix. This is works extremely well now.
Let’s see the same process like the OP to compare it.
Don:
When the chat channel retention days
and/or chat dm retention days
set to 0
it means retain messages forever. But now on (channel / personal) retention day messages it seems like it saved for 0
day. Now it detects correctly if the value is 0
.
Thanks again!
إعجابَين (2)
j.jaffeux
(Joffrey Jaffeux)
16 فبراير 2023، 12:13ص
7
Perfect, thanks for checking, very much appreciated
إعجابَين (2)
j.jaffeux
(Joffrey Jaffeux)
تم إغلاقه في
20 فبراير 2023، 7:00ص
8
This topic was automatically closed after 4 days. New replies are no longer allowed.