shimoju / metabase-ruby

Ruby client library for Metabase API
MIT License
28 stars 27 forks source link

Add params arg #13

Closed shimoju closed 6 years ago

shimoju commented 6 years ago

GET /api/cardはquery stringでオプションを指定できるので、params引数を追加しました。 https://github.com/metabase/metabase/blob/master/docs/api-documentation.md#get-apicard

client.cards({ f: 'archived' })のように使えます。 キーワード引数で受け取れるようにしてもいいけど、Metabase側のパラメータが追加・変更されるたびに対応が必要なのが面倒です。 最後の引数がハッシュのときは{}を省略できる仕様を使えば、client.cards(f: 'archived')と書けるのでそれでいいかなと。(キーワード引数の導入前によく使われていたやつ)

また、すべてのメソッドにparams引数を追加するようにします。 GET /api/user/currentなど指定できるオプションがないものもありますが、

という理由。

loginではusername,passwordが引数で指定されればそれを、なければ@username,@passwordを使用するようにしました。