nulab / backlog-js

Backlog API version 2 client for browser and node.
https://developer.nulab-inc.com/docs/backlog/
33 stars 13 forks source link

Typescript3 #3

Closed netebakari closed 4 years ago

netebakari commented 6 years ago

はじめまして。Backlogを公私で利用している者です。 Node.js(AWS Lambda)からBacklogのAPIを叩くためにbacklog-jsを使おうとしたのですが、 ・古いモジュールを参照しているのでnpmが警告を出す ・TypeScript3でうまくビルドできない ・GETで呼ぶべきところでDELETEを呼んでいるメソッドがある ・https://developer.nulab-inc.com/ja/docs/backlog/ に記載されているAPIのうち一部が実装されていない という問題を見つけたため、取り急ぎこれらを修正してpull requestをお送りします。

#Git初心者のため何か不始末がありましたらご容赦ください……

なおBacklogのAPI一覧とその並び順は https://developer.nulab-inc.com/ja/docs/backlog/ から取得し、Google spreadsheetに出力してあります。 https://docs.google.com/spreadsheets/d/1-6UaL2uoodztaSXuA1aGwCd8jOm3qLnBYbYR9OuC8xk/edit?usp=sharing

よろしくお願いします。

mmktomato commented 4 years ago

@netebakari プルリクエストありがとうございます。 長い期間放置して申し訳ありません。

現在こちらのプルリクエストは master とコンフリクトするため取り込めない状態です。 今からコンフリクトを解消していただくのも申し訳ないので、こちらのコミットを cherry-pick してコンフリクトを解消したブランチとプルリクエストを用意しました。

https://github.com/nulab/backlog-js/pull/10

もしよろしければ上記のプルリクエストをマージさせていただいて、こちらのプルリクエストはクローズさせていただければと思いますが、よろしいでしょうか? (その場合でもコミットの Author は netebakari 様となるため、コントリビューションは記録されます。)

mmktomato commented 4 years ago

@netebakari 上記の通り、こちらクローズさせて頂きます。プルリクエストを頂きありがとうございました。