search

toolbox-team / reddit-moderator-toolbox-legacy

LEGACY VERSION do not use
http://www.reddit.com/r/toolbox
Apache License 2.0
66 stars 40 forks source link

Enabled multireddit for modbar #836

Closed phornicaite closed 7 years ago

phornicaite commented 7 years ago

Tried switching to a multireddit instead of /r/mod for performance reasons, and found out that there was no support for actual multis. This fixes that with very minimal changes, although it does not look the cleanest.

This allows for public multi's such as /user/awkwardtheturtle/m/cat, or a users own multi's, such as /me/m/realsubs

eritbh commented 7 years ago

Actual multireddits don't have modqueue pages, do they? I don't follow how this works.

creesch commented 7 years ago

As @Geo1088 said, I am also a tad confused by this. Did you test it?

creesch commented 7 years ago

Turns out they have queues and all that. I am going to check out the code to see how it works out.

creesch commented 7 years ago

Okay, I am going to close this pull request and open up an issue for it. As this code will break a bit too much and doesn't contain migration either.