Skip to content
This repository has been archived by the owner on Jun 8, 2023. It is now read-only.

Handle empty url case for no data from bing #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Handle empty url case for no data from bing #13

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
3 changes: 3 additions & 0 deletions lib/bing/ads/api/v12/services/reporting.rb
View file
Open in desktop
Expand Up @@ -35,6 +35,9 @@ def report_url(report_request_id)
end

def report_body(report_request_id)
report_url = report_url(report_request_id)
return if report_url.nil?

HttpClient.download(report_url(report_request_id))
end

Expand Down