search

bellroy / lesswrong

Less Wrong platform
http://lesswrong.org/
Other
45 stars 23 forks source link

disable user posts to main #556

Closed xrpd closed 8 years ago

xrpd commented 8 years ago

This is intended to resolve issue #549

can_submit for main now always returns False for non-admins. Because of this, main is disabled as a submit option in the drop down menu in newlink.html. I changed the reason to "(Posting to main is currently disabled)" instead of "(Your karma must be at least 40 to post here)."

I tested in the vagrant vm using admin, high karma and low karma users.

wezm commented 8 years ago

Have you looking into how this interracts with editing?

xrpd commented 8 years ago

Have you looking into how this interacts with editing?

Admins and moderators can always edit articles in main, but not normal users - not even article authors. I'll ask if this acceptable, and update the PR if not.

xrpd commented 8 years ago

@wezm, @vaniver indicated interest in allowing user edits to to existing posts in the main subreddit. I added a new PR #557 that includes that functionality. I made it a new PR, since it seems to me that a policy debate could still possibly favor #556. If #557 is accepted, #556 should be rejected.

vaniver commented 8 years ago

Agree that #557 is the desired option instead of #556. (I haven't tested #557 myself.)

cdaloisio commented 8 years ago

Closing this PR in favour of #557