Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 5649449867018240: #664 - Increase The Mention Limit (Closed)

Created:
July 22, 2014, 3:13 a.m. by mathias
Modified:
July 28, 2014, 1:39 a.m.
Visibility:
Public.

Description

See https://issues.adblockplus.org/ticket/664

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M modules/discourse/manifests/init.pp View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 3
mathias
July 22, 2014, 3:13 a.m. (2014-07-22 03:13:58 UTC) #1
Felix Dahlke
Looks good. I'd suggest we pick 100 as a limit here, since that's a number ...
July 22, 2014, 4:56 a.m. (2014-07-22 04:56:16 UTC) #2
Wladimir Palant
July 22, 2014, 6:55 a.m. (2014-07-22 06:55:49 UTC) #3
LGTM

Having 100 as the limit would be fine with me as well but I don't see 50 as an
issue - even if we have 50 employees in a few years, having to mention all of
them in a Discourse post sounds extremely unlikely and a bad idea in the first
place.

Powered by Google App Engine
This is Rietveld