blinktrade / frontend-legacy

GNU General Public License v3.0
121 stars 182 forks source link

Server isn't returning a JSONP response #236

Closed Rodrigora closed 6 years ago

Rodrigora commented 6 years ago

Hello Blinktrade,

I'm following your documentation and I've found this error when trying to access the public API:

import { BlinkTradeRest } from 'blinktrade';
const BlinkTrade = new BlinkTradeRest({
    currency: 'BRL',
    prod: false
  })
BlinkTrade.ticker().then(console.log)

screen shot 2018-01-02 at 23 41 23

Look like the library is requiring a JSONP response, but the server returns a JSON response. Is it a bug in the library/server?

pinhopro commented 6 years ago

The server doesn't support jsonp

Rodrigora commented 6 years ago

Hi @pinhopro, thanks for your reply.

Yes, I see.

So I don't understand why the Blinktrade library is requesting a JSONP format. I'll try again, thanks!

cesardeazevedo commented 6 years ago

Hi @Rodrigora

We deprecated jsonp and we already ship a fix on the library (0.0.17), take a look at this issue for a better understanding (https://github.com/blinktrade/BlinkTradeJS/issues/32) and just update the library.

Rodrigora commented 6 years ago

Thanks guys! And sorry for creating this issue in the wrong repo On Tue, 9 Jan 2018 at 23:45 César D Azevedo notifications@github.com wrote:

Hi @Rodrigora https://github.com/rodrigora

We deprecated jsonp and we already ship a fix on the library (0.0.17), take a look at this issue for a better understanding ( blinktrade/BlinkTradeJS#32 https://github.com/blinktrade/BlinkTradeJS/issues/32) and just update the library.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/blinktrade/frontend/issues/236#issuecomment-356472776, or mute the thread https://github.com/notifications/unsubscribe-auth/AAWpbQsV__rQAc4ex-75yliYVQrBsrYUks5tJBYrgaJpZM4RRRXz .