Don
10 Dicembre 2022, 12:44pm
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 Mi Piace
piffy
9 Febbraio 2023, 5:36am
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 Febbraio 2023, 10:56pm
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 Mi Piace
j.jaffeux
(Joffrey Jaffeux)
15 Febbraio 2023, 1:50pm
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 Mi Piace
Don
15 Febbraio 2023, 5:09pm
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 Mi Piace
j.jaffeux
(Joffrey Jaffeux)
16 Febbraio 2023, 12:13am
7
Perfect, thanks for checking, very much appreciated
2 Mi Piace
j.jaffeux
(Joffrey Jaffeux)
Chiuso
20 Febbraio 2023, 7:00am
8
This topic was automatically closed after 4 days. New replies are no longer allowed.