Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Threaded modmail parameter #1454

Closed

Conversation

13steinj
Copy link
Contributor

@13steinj 13steinj commented Nov 1, 2015

This allows one to get modmail in it's threaded form via the API, which can be useful depending on how the user works.

(Of course, without having to constantly enable the user preference if they for whatever reason want flat modmail)

I've also added two different commits for a GUI to select threaded modmail via the browser. If you'd like the first, just disregard the second. If you'd like the second, squash the third commit with the second. If you'd like neither, disregard both :P.

E: of course, the line in example.ini would have to be moved to reddit's run.ini

This allows one to get modmail in it's threaded form via the API, which can be useful depending on how the user works.
@umbrae
Copy link
Contributor

umbrae commented Nov 11, 2015

Thanks for the contribution - I think we need to close this so that we can have a larger discussion about threaded modmail in general, though.

Also just as an aside this isn't the way we intend for feature flags to be used: they're typically for short-to-medium term things that need to go out to a segment of people. In this case we'd need to keep this feature flag around forever, which would have deleterious effects on code quality and also having a bunch of flags like that ends up being not great for zookeeper performance.

@umbrae umbrae closed this Nov 11, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants