#2001 - Make chat message back-log configurable
-
By Chris Graham
- Added
- 3 views
| Identifier | #2001 |
|---|---|
| Issue type | Feature request or suggestion |
| Title | Make chat message back-log configurable |
| Status | Completed |
| Handling member | Chris Graham |
| Addon | chat |
| Description | We have
define('CHAT_BACKLOG_TIME',60*5); // 5 minutes of messages if you enter an existing room Make it a config option. |
| Steps to reproduce | |
| Funded? | No |
The system will post a comment when this issue is modified (e.g., status changes). To be notified of this, click "Enable comment notifications".
Comments
This should be a config option too.
I think the backlog-time is actually completely ignored. We need to decide whether to just remove that, or implement both options. I verge on just having the static number, as it is safer performance-wise, and I don't think limiting by date is actually that useful (so what if old stuff shows? the maximum quantity is what matters).
I agree strongly with this. Even if you set it to say, one day, if say a chat room exploded with activity within the last 24 hours, that'll be a big performance hit as the chat loads all those messages.
It is now as you want. There's a "Chat messages to show" option, and it's not time based.