Skip to content

Conversation

Copy link

Copilot AI commented Oct 2, 2025

Summary

This PR updates the author submission email templates to provide clearer guidance to authors about how to get support for their submissions, addressing the issue that authors were replying to the unmonitored admin@theoj.org inbox.

Changes Made

The author submission email now includes:

  1. Prominent warning not to reply - Both HTML and text versions now include a clear notice that admin@theoj.org is unmonitored and authors should not reply to this email
  2. Clear support channel information - Authors are informed that a GitHub issue will be opened after the initial desk check, and all questions and support should be directed there
  3. Timing expectations - The existing language about papers potentially taking "a number of weeks" during busy periods was retained

Updated Email Content

The new warning message reads:

Please do not reply to this email. This email address (admin@theoj.org) is unmonitored. Once your paper has completed an initial desk check, a GitHub issue will be opened for your submission. All questions and support for your paper should be directed to that GitHub issue.

Files Modified

  • app/views/notifications/author_submission_email.html.erb - Added bold warning text with proper HTML formatting
  • app/views/notifications/author_submission_email.text.erb - Added corresponding plain text warning
  • spec/mailers/notifications_spec.rb - Added test coverage to verify the warning message is present

Preview

Author Submission Email Preview

The screenshot shows both the HTML and text versions of the updated email, with the warning message clearly visible.

Testing

Added automated test to ensure the warning message is present in the email body:

  • Verifies "Please do not reply to this email" text is included
  • Verifies mention of admin@theoj.org being unmonitored
  • Verifies information about GitHub issue being opened

Fixes the issue where authors were attempting to get support by replying to the unmonitored submission confirmation email.

Original prompt

This section details on the original issue you should resolve

<issue_title>Author submission email should have more information for authors</issue_title>
<issue_description>We should point out to authors that they should not respond to the email (admin@theoj.org) as the inbox is unmonitored, and that support for their paper should come via the issue on GitHub that will be opened once a first 'desk check' of their paper has happened.

Finally, we should update our language to make it clear that they might not see their paper show up on GitHub for a few weeks, especially if we're busy.</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #1461

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits October 2, 2025 12:06
…r submission emails

Co-authored-by: arfon <4483+arfon@users.noreply.github.com>
Co-authored-by: arfon <4483+arfon@users.noreply.github.com>
Copilot AI changed the title [WIP] Author submission email should have more information for authors Update author submission email to clarify unmonitored inbox and support channels Oct 2, 2025
Copilot AI requested a review from arfon October 2, 2025 12:11
Copilot finished work on behalf of arfon October 2, 2025 12:11
@arfon arfon marked this pull request as ready for review October 2, 2025 13:33
@danielskatz
Copy link
Collaborator

This is scary

@arfon
Copy link
Member

arfon commented Oct 3, 2025

This is scary

The edits, or the fact that Copilot made the changes?

@danielskatz
Copy link
Collaborator

The latter. I also wonder if just changing the email is sufficient or if there are other things that also should be changed.

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.

Author submission email should have more information for authors

3 participants