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.

Sign up for GitHub

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

Jump to bottom

Change http to https in notify_moderators_of_comment_spam.html.erb #8631

Closed
noi5e opened this issue Oct 20, 2020 · 2 comments · Fixed by #8593
Closed

Change http to https in notify_moderators_of_comment_spam.html.erb #8631

noi5e opened this issue Oct 20, 2020 · 2 comments · Fixed by #8593
Labels
first-timers-only They need to be well-formatted using the First-timers_Issue_Template.

Comments

@noi5e
Copy link
Contributor

noi5e commented Oct 20, 2020

🔊 NOTE: (This issue is a part of Issue #8587 and has been claimed at the time of writing this issue. However, there are several places in the codebase where this change needs to happen, and issues haven't been made for each instance yet. If you are a first-time contributor, please refer to #8587 for more info on how to find instances of 'http' in the codebase. From there, you can make your own FTO issue and pull request to solve it.)

Please see #8587 for more context behind this issue.
@noi5e noi5e added the first-timers-only They need to be well-formatted using the First-timers_Issue_Template. label Oct 20, 2020
@noi5e
Copy link
Contributor Author

noi5e commented Oct 20, 2020

@cesswairimu @jywarren, please assign @amanda-ariyaratne to this issue. They have an open PR at #8593.

I decided not to type up extensive issue notes if the contributor already has an open PR, I thought it would be overkill since it seems as if they don't need the help.

@amanda-ariyaratne
Copy link
Contributor

Hello, I have opened a pull request to this issue linking the issue #8587.
It is awaiting review.

@cesswairimu cesswairimu mentioned this issue Nov 23, 2020
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
first-timers-only They need to be well-formatted using the First-timers_Issue_Template.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Changed http to https in notify_moderators_of_comment_spam.html.erb (#8587) amanda-ariyaratne/plots2
2 participants
@noi5e @amanda-ariyaratne