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

GET, POST & DELETE の各メソッドに対応するswitch文を実装しました 2 #25

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nyeoro-bong
Copy link
Collaborator

3−14HTTPのメソッドの実装/変更内容

  • リクエストメソッドに対応したサーバーレスポンスを実装
  • エラー発生時のサーバーレスポンスを実装

実装/変更理由

  • httpリクエストのメソッドGET, POST, DELETE とそれ以外(default)の場合でswitch文で分岐処理
  • 各メソッドごとのレスポンス内容を記述

スクリーンショット/デザイン

スクリーンショット 2021-10-20 21 06 34

スクリーンショット 2021-10-20 21 05 22

確認に必要なコマンド群

docker-compose up -d
docker-compose exec app bash
tmux
node index.js
curl -d 'message=おはこんばんちわ' http://localhost:8000/messages
curl -X DELETE http://localhost:8000/messages

その他

  • ほめられていた ”りさりさと” さんのお作法に乗っかってコメント作成しました。
  • デビュープルリクでいろいろ過不足や勘違い等あるかと思います。やんわりとご指導いただけるとありがたいです。スパルタ式だとすぐ(´ω`)しちゃうタイプです。
    P.S.
  • 先々週の雨の金曜日からタチの悪いカゼをこじらせてました。今日やっと復活してきました。
  • すぐお返事頂いていたのにずっと音無しで大変申し訳ありませんです🙇‍♂️

@satsukizzz
Copy link
Owner

おかえりなさい!大丈夫かなと心配してました。
新しいプルリク出してもらって助かります。これでレビューしていきます〜👍👍

@satsukizzz satsukizzz changed the title Nyoro 314 GET, POST & DELETE の各メソッドに対応するswitch文を実装しました 2 Nov 1, 2021
Copy link
Collaborator

@HappyKumamoto HappyKumamoto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

動作確認できました。

3-14

↑やっとblanchに入ることができて嬉しくてパチリ

Copy link
Collaborator

@HappyKumamoto HappyKumamoto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

この変更を承認します。

Copy link
Owner

@satsukizzz satsukizzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コードいいと思います!!
approveします 👍

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.

3 participants