AO3-4532 Fixing it so Abuse reports go through. Removing language lis… #2460
3 participants
app/models/feedback_reporters/feedback_reporter.rb
| @@ -32,6 +32,20 @@ def send_report! | ||
| ) | ||
| end | ||
| + def send_abuse_report! | ||
| + HTTParty.post("#{ArchiveConfig.ABUSE_REPORTS_SITE}/projects/#{project_id}/bugs", | ||
| + headers: { | ||
| + "Content-Type" => "application/xml", | ||
| + "Accept" => "application/xml" | ||
| + }, | ||
| + basic_auth: { | ||
| + username: ArchiveConfig.ABUSE_REPORTS_USER, | ||
| + password: ArchiveConfig.ABUSE_REPORTS_PASSWORD | ||
| + }, | ||
| + body: xml | ||
| + ) |
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
|
app/models/feedback_reporters/feedback_reporter.rb
| @@ -32,6 +32,20 @@ def send_report! | ||
| ) | ||
| end | ||
| + def send_abuse_report! | ||
| + HTTParty.post("#{ArchiveConfig.ABUSE_REPORTS_SITE}/projects/#{project_id}/bugs", | ||
| + headers: { |
|
Align the parameters of a method call if they span more than one line.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
|
houndci-bot
and 1 other
commented on an outdated diff
app/models/abuse_report.rb
| @@ -55,15 +55,15 @@ def email_and_send | ||
| end | ||
| def send_report | ||
| - return unless %w(staging production).include?(Rails.env) | ||
| + #return unless %w(staging production).include?(Rails.env) |
|
I think you just commented this out for testing on your machine... we still want it in
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
|
Issue: https://otwarchive.atlassian.net/browse/AO3-4569
Also, were you going to fix https://otwarchive.atlassian.net/browse/AO3-4568?
scottsds
added Coder has actioned review and removed Reviewed: Needs Coder Action labels
scottsds
added
some commits
May 15, 2016
|
|
scottsds |
Some checks were not successful
Details
continuous-integration/travis-ci/pr
— The Travis CI build could not complete due to an error
hound
— 2 violations found.
|
280b3f0
|
||
|
|
scottsds |
Some checks were not successful
Details
continuous-integration/travis-ci/pr
— The Travis CI build could not complete due to an error
hound
— 2 violations found.
|
f56701e
|
||
|
|
scottsds |
b4f397b
|
|||
|
|
scottsds |
d15d22f
|
2 checks passed
Details
continuous-integration/travis-ci/pr
The Travis CI build passed
hound
2 violations found.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
…ting from reports new page
Issue: https://otwarchive.atlassian.net/browse/AO3-4569
https://otwarchive.atlassian.net/browse/AO3-4570
https://otwarchive.atlassian.net/browse/AO3-4568