Skip to content
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

Rescreen comment edits #1429

Merged
merged 1 commit into from Jun 25, 2015
Merged

Rescreen comment edits #1429

merged 1 commit into from Jun 25, 2015

Conversation

woggy
Copy link
Contributor

@woggy woggy commented Jun 25, 2015

For entries which screen comments, editing a comment which had been
unscreened would immediately display the edited comment. This patch
changes the behavior such that an edited comment is immediately
rescreened, for the post owner(s) to review the changed content and
determine whether it is still eligible for unscreening. The user
will be warned that their edited comment is being rescreened.

Fixes #1230

For entries which screen comments, editing a comment which had been
unscreened would immediately display the edited comment. This patch
changes the behavior such that an edited comment is immediately
rescreened, for the post owner(s) to review the changed content and
determine whether it is still eligible for unscreening.  The user
will be warned that their edited comment is being rescreened.

Fixes dreamwidth#1230
afuna added a commit that referenced this pull request Jun 25, 2015
@afuna afuna merged commit cc74020 into dreamwidth:develop Jun 25, 2015
@afuna
Copy link
Member

afuna commented Jun 25, 2015

knowing where to apply the hammer....

@woggy woggy deleted the 1230-screenedit branch June 25, 2015 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants