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

Script that sets blockedReason and blocks reply rquests #267

Merged
merged 8 commits into from Nov 24, 2021

Conversation

MrOrz
Copy link
Member

@MrOrz MrOrz commented Nov 18, 2021

Implements part of the blocking mechanism (ReplyRequest only) designed as the 2nd milestone in the User Blocking feature.

  • A script, blockUser, is added to set blocked reason (expected to be an URL to an announcement) and changes the user's normal replyRequest status
  • Update article's reply request count & last replied at using the remaining normal replyRequests
  • The scripts can be executed multiple times in case of error
    • It is tested using production snapshot data; did not encounter any error.

@MrOrz MrOrz self-assigned this Nov 18, 2021
@coveralls
Copy link

coveralls commented Nov 18, 2021

Coverage Status

Coverage decreased (-0.1%) to 86.178% when pulling 6b7d0e2 on block-user-script into b4118e9 on scripts-utils.

@MrOrz MrOrz changed the base branch from master to scripts-utils November 20, 2021 05:20
@MrOrz MrOrz changed the title Block user script Script that sets blockedReason and blocks reply rquests Nov 21, 2021
@MrOrz MrOrz marked this pull request as ready for review November 22, 2021 05:19
Base automatically changed from scripts-utils to master November 24, 2021 18:43
@MrOrz MrOrz merged commit 4719a8a into master Nov 24, 2021
@MrOrz MrOrz deleted the block-user-script branch November 24, 2021 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants