Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Typescript3 #3

Closed
wants to merge 5 commits into from
Closed

Typescript3 #3

wants to merge 5 commits into from

Conversation

netebakari
Copy link
Contributor

はじめまして。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
Copy link
Member

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

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

#10

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

@mmktomato
Copy link
Member

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

@mmktomato mmktomato closed this Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants