Lets see if this fixes yahoos inability to respond to a legit query #52

Merged
merged 2 commits into from Dec 19, 2016

Conversation

2 participants
Owner

CrazyLemon commented Dec 19, 2016

No description provided.

@CrazyLemon CrazyLemon requested a review from zdobersek Dec 19, 2016

- cb "#{(100 / (212 - 32) * (item.condition.temp - 32)).toFixed(2)}°C #{item.link}"
+ if res?
+ item = res.channel.item
+ cb "#{(100 / (212 - 32) * (item.condition.temp - 32)).toFixed(2)}°C #{item.link}"
@zdobersek

zdobersek Dec 19, 2016

Contributor

212 - 32 = 180.

@CrazyLemon

CrazyLemon Dec 19, 2016

Owner

You so smart! Thats why i assigned you!

@zdobersek

zdobersek Dec 19, 2016

Contributor

Then y u no fix it.

scripts/vreme.coffee
+ item = res.channel.item
+ cb "#{(100 / (212 - 32) * (item.condition.temp - 32)).toFixed(2)}°C #{item.link}"
+ else
+ cb "Podatka o vremenu ni ... ker yahoo sucks"
@zdobersek

zdobersek Dec 19, 2016

Contributor

Yahoo*

s/yahoo/Yahoo
Coz @zdobersek is a yahoo
Contributor

zdobersek commented Dec 19, 2016

Oh, lih sem se spomnu -- meh mi je.

@zdobersek zdobersek merged commit c675e3e into master Dec 19, 2016

@dz0ny dz0ny deleted the fix/vreme-test branch Jan 3, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment