Skip to content

Connect with uhn test server. - #6

Open
wichopy wants to merge 2 commits into
masterfrom
connect-uhn-test-server
Open

Connect with uhn test server.#6
wichopy wants to merge 2 commits into
masterfrom
connect-uhn-test-server

Conversation

@wichopy

@wichopy wichopy commented Jul 3, 2018

Copy link
Copy Markdown
Owner

Connect to uhn test server and query all questionnaire responses.

@ghost ghost assigned wichopy Jul 3, 2018
@ghost ghost added the in progress label Jul 3, 2018
@wichopy
wichopy requested a review from ronniemara July 3, 2018 03:28
Comment thread connector.js Outdated
return fetch(serverUrl + '/fhir/' + resource + '?' + searchParams)
const url = serverUrl + resource + (searchParams ? '?' + searchParams : '')
return fetch(url)
.then(res => res.json())

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you log to console the response status, just as a sanity check e.g. console.log(res.status)

Comment thread resolvers.js Outdated
return result;
})
},
// args: linkId

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use jsdocs, they are self-documenting

@wichopy

wichopy commented Jan 25, 2019

Copy link
Copy Markdown
Owner Author

@ronniemara

Example of console logs:

Uploading Screen Shot 2019-01-25 at 9.11.14 AM.png…

Also working with live data :D

@wichopy
wichopy changed the base branch from 4-questionnaire-response-reducer to master January 25, 2019 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants