-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FEATURE: New post editing period for >= tl2 users #8070
Conversation
rimian
commented
Sep 4, 2019
- Adds a new setting and for tl2 editing posts (30 days same as old value)
- Sets the tl0/tl1 editing period as 1 day
* Adds a new setting and for tl2 editing posts (30 days same as old value) * Sets the tl0/tl1 editing period as 1 day
You've signed the CLA, rimian. Thank you! This pull request is ready for review. |
* Fix site setting on guardian spec
Is there a relevant meta discussion for this change? I'm looking for some context here. |
Sorry, I am not sure where to reference things and where to put the context just yet. I'll clear that up before my next PR :) |
Yeah a quick link in the PR is always a good idea cause there are many of us, but really not a huge deal. |
Overall the change looks good to me! minor changes to the specs and we should be good to merge. |
* Avoid shared examples * Use update_columns to avoid callbacks on user during tests
This pull request has been mentioned on Discourse Meta. There might be relevant details there: https://meta.discourse.org/t/spam-bots-tricking-discourse-filter-by-editing/101684/23 |