Skip to content

Conversation

@not-varram
Copy link

Pull Request Checklist

Issue

https://otwarchive.atlassian.net/browse/AO3-6198

Purpose

What does this PR do?

This PR clarifies Akismet configuration and ensures spam checking only runs in staging/production environments.

Changes made:

  1. Replaces AKISMET_KEY and AKISMET_NAME values in config.yml with empty strings to make it clear they're not actual valid credentials
  2. Updates check_for_spam methods in Comment, Feedback, and AbuseReport models to only execute in staging and production environments (matching the existing pattern in the Work model)

This prevents spam checking from running in development/test environments where Akismet credentials aren't configured, while maintaining existing functionality in staging and production.

Credit

What name and pronouns should we use to credit you in the Archive of Our Own's Release Notes?

varram (he/him)

P.S. This is my first PR (planning on doing many more!) so please let me know if I need to do anything else for this :).
Also Here is my jira account (for permissions? 🤞)

@sarken
Copy link
Collaborator

sarken commented Jan 13, 2026

Hi, varram!

Thank you so much for this pull request. Someone will be along to review it soon.

Unfortunately, Jira has been unreliable about showing us new accounts in the admin panel lately, and the link you provided says it's for a user that doesn't exist. If you could either reply here or send an email to otw-coders@transformativeworks.org with your email address, we'll make sure the account gets set up right. (Once it's set up, you'll have the ability to comment on, assign, and transition issues.)

While we get that straightened out, I've updated the Jira issue status to In Review so no one mistakenly creates a duplicate pull request.

Thanks again for contributing! If you have any questions, you can contact us at the same email address listed above.

@not-varram
Copy link
Author

Hi @sarken !

Thanks for the quick reply!

Here’s the email address associated with my Jira account: email@varram.me.

Appreciate you moving the issue to In Review while this gets sorted out.

Please let me know if you need anything else from me.

Cheers,
varram

@sarken
Copy link
Collaborator

sarken commented Jan 13, 2026

Your Jira permissions should be all set the next time you log in!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants