search

xpjp / xpfiat-bot

XP JPのディスコード上で動作しているボットです。 XPをJPYに変換したりする機能があります。
MIT License
20 stars 5 forks source link

read_price_from_jsonでエラーが出ていたのを修正しました。 #55

Closed chipstar closed 6 years ago

chipstar commented 6 years ago

何を解決するのか

read_price_from_jsonで下記エラーが出ていたのを修正しました。 Exception: #

直接コードを書き換えてすでにデプロイ済みです。

chipstar commented 6 years ago

@xpjp/ruby-reviewer レビューお願いしますー

toshikidoi commented 6 years ago

https://github.com/xpjp/xpfiat-bot/pull/51 で私がPR出していたのですが、既にこちらの内容で本番を直接修正されたということで、こちらを残しましょうか😄

1点だけ〜。 dogeメソッドの中でread_priceが呼ばれているので、後で混乱しないように、dogeメソッド内のto_fを消して頂いても良いでしょうか🙇 あと、メソッド名的に、個人的にはto_fすべきなのはread_price_from_json内ではなくread_price内かなと思いましたが、この件はどちらでもOKです〜。

chipstar commented 6 years ago

@toshikidoi アドバイスありがとうございます！read_priceでto_fするように変更しました dogeメソッドのto_fも削除済みです。 @xpjp/ruby-reviewer 再度レビューお願いします！