Chicago / lead-safe-api-docs

http://dev.cityofchicago.org/docs/lead-safe/
1 stars 1 forks source link

Response JSON file includes escape characters #44

Closed tomschenkjr closed 6 years ago

tomschenkjr commented 6 years ago

As part of testing, @AOCjcarr informs us:

We are currently receiving:

{
"items": [
{
"result": "{\"risk_score_notes\": \"1. Please follow protocol for lead safety.\", \"errors\": [], \"network_id\": \"AllianceChicago\", \"timestamp\": \"2018-05-08 15:19:05\", \"clinic_id\": \"NN\", \"patient_id\": \"240049\", \"risk_score\": \"0.182835714286\", \"version\": \"1.0.0-rc.3\", \"location_id\": \"NNHS\"}"
}
]
}

@geneorama - can you replicate this? Do you see the same in the response documents?

geneorama commented 6 years ago

I'm not sure when the escapes appear. I don't see them when I call the python code directly. I do see them in R when I get the response. I'll keep working on it and work with George.

geneorama commented 6 years ago

I'm pretty sure this is a single issue related to the quoting mentioned in #45.

I'm closing this for now, but we can reopen if it turns out to be a separate issue.

tomschenkjr commented 6 years ago

Let's keep this open so Sun can remove the backslashes.

AOCjcarr commented 6 years ago

Correct, this was resolved with the quoting fix

Thanks, Jeremy Carr

From: Gene Leynes notifications@github.com Sent: Thursday, May 17, 2018 9:27 AM To: Chicago/lead-safe-api-docs lead-safe-api-docs@noreply.github.com Cc: Jeremy Carr jcarr@alliancechicago.org; Mention mention@noreply.github.com Subject: Re: [Chicago/lead-safe-api-docs] Response JSON file includes escape characters (#44)

I'm pretty sure this is a single issue related to the quoting mentioned in #45https://github.com/Chicago/lead-safe-api-docs/issues/45.

I'm closing this for now, but we can reopen if it turns out to be a separate issue.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/Chicago/lead-safe-api-docs/issues/44#issuecomment-389885945, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AenUxwcYskj0EFqNpGApyH1F5vIrWuCcks5tzYjQgaJpZM4T5WFM.

geneorama commented 6 years ago

Thanks Jeremy for confirming! Closing now.