Fix Issue 50: connection.js remove onlyJsonResponse

This commit is contained in:
michielmulders 2017-08-15 21:15:13 +02:00 committed by Tim Daubenschütz
parent 484a3eb608
commit 5ec5df7e1d
2 changed files with 4 additions and 7 deletions

View File

@ -89,9 +89,8 @@ export default class Connection {
* @public
* @param publicKey
* @param spent
* @param onlyJsonResponse
*/
listOutputs(publicKey, spent, onlyJsonResponse = true) {
listOutputs(publicKey, spent) {
const query = {
public_key: publicKey
}
@ -102,7 +101,7 @@ export default class Connection {
}
return this._req(this.getApiUrls('outputs'), {
query
}, onlyJsonResponse)
})
}
/**
@ -159,7 +158,6 @@ export default class Connection {
/**
* @public
*
* @param transaction
*/
postTransaction(transaction) {
@ -172,7 +170,6 @@ export default class Connection {
/**
* @public
*
* @param search
*/
searchAssets(search) {

View File

@ -12,7 +12,7 @@ const DEFAULT_REQUEST_CONFIG = {
* Small wrapper around js-utility-belt's request that provides url resolving,
* default settings, and response handling.
*/
export default function request(url, config = {}, onlyJsonResponse = true) {
export default function request(url, config = {}) {
// Load default fetch configuration and remove any falsy query parameters
const requestConfig = Object.assign({}, DEFAULT_REQUEST_CONFIG, config, {
query: config.query && sanitize(config.query)
@ -30,7 +30,7 @@ export default function request(url, config = {}, onlyJsonResponse = true) {
}
return baseRequest(apiUrl, requestConfig)
.then(res => (onlyJsonResponse ? res.json() : { json: res.json(), url: res.url }))
.then(res => (res.json()))
.catch(err => {
console.error(err)
throw err