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

Implement posting age requirement #308

Merged
merged 4 commits into from
Sep 28, 2023
Merged

Implement posting age requirement #308

merged 4 commits into from
Sep 28, 2023

Conversation

zwolf
Copy link
Member

@zwolf zwolf commented Sep 27, 2023

Adds the length of time since a user's created_at timestamp as a policy check to determine whether that user can post new comments. Direct result of spam issues on Talk.

This requires a new env var, 'POSTING_AGE_REQUIREMENT', to exist in order to be checked. This is a string that represents the number of hours old an account has to be to be allowed to post a comment on specifically the 'zooniverse' board. Default is 24.

Before merging this PR and deploying this feature, we should have a follow up conversation.

Copy link
Collaborator

@yuenmichelle1 yuenmichelle1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zwolf zwolf merged commit c1886b0 into master Sep 28, 2023
1 check passed
@zwolf zwolf deleted the posting-age-requirement branch September 28, 2023 18:19
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.

3 participants