apiRoute作成 #12
Merged
apiRoute作成 #12
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Closed
|
ありがとうございます！ |
kei-nishikawa48
approved these changes
Mar 27, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
#8
今回の PR で行ったこと
apiRouteの作成
/api/lyricsSearch-[query].ts
/api/youtubeSearch-[query].ts
getやpostの変数が現状はとても適当です。
とりあえず渡せます、とだけ実装しました。
テスト用にapiを呼び出す画面を作成(開発が進むと削除するもの)
api.test.svelte
動作確認(どのような動作確認を行ったのか？ 結果はどうか？)
thunder clientとapi.test.svelteで、apiを叩いて呼び出されることを確認
影響範囲(行った作業によってどこまで影響が及ぶようになるか)
現状は特になし
実装するにあたって参考にした URL
https://kit.svelte.dev/docs/routing#endpoints-body-parsing
sveltejs/kit#3384
sveltejs/kit#3777
確認して欲しいこと
コーディング内容がissueのやりたいことと合っているか。
コーディング内容が、TypeScript的に問題ないか。
コーディングが適当すぎないか。
懸念点
apiRouteのrequestとかのTypeScriptの型は「自動でつけるので自分でつけなくていい」的なことが書いてあるようなので付けませんでした。
https://kit.svelte.dev/docs/types#sveltejs-kit-requesthandler
他にもひとまずエラーにならなかったので型をつけていない部分もあるので、問題がありましたらお手数ですがご指摘お願い致します。
よろしくお願い致します。