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

feat:mockサーバーを立ち上げるコマンドを作成 #69

Merged
merged 4 commits into from
Nov 25, 2024

Conversation

mathsuky
Copy link
Collaborator

やったこと

  • docker経由でmockサーバーを立ち上げること
  • 基本的なレスポンスに対してmockデータを返すようにしたこと

やっていないこと

  • ログイン機構の実現(メアドログイン周りはデータ返却だけ実現,OAuthはノータッチ,各種mockAPIも認証による保護なし)
  • クエリパラメタによる返却データの複雑な制御(ソートなど)

クエリパラメタによるソートは余裕があれば今後実装したいが,まず先に基本的な部分を開発で利用できるようにしたい。

@mathsuky
Copy link
Collaborator Author

@ZOI-dayo
mock関連の設定を追加した関係でAPI関係のファイルの生成場所やgitignoreの設定も一部変更していますので,ご確認ください。

Copy link
Collaborator

@ZOI-dayo ZOI-dayo left a comment

Choose a reason for hiding this comment

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

ありがとうございます! 偶然小さなバグ?を見つけたのでコメントを付けています。
Dockerfilemock_test.jsondemo.jsonの3ファイルが不要な気がするのですが、これらは何かに使われていますか...?

src/api/mock/front-back-api-mock.json Outdated Show resolved Hide resolved
@mathsuky
Copy link
Collaborator Author

レビューありがとうございます!修正したので,再レビューお願いいたします

Copy link
Collaborator

@ZOI-dayo ZOI-dayo left a comment

Choose a reason for hiding this comment

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

ありがとうございます! 一点だけ気になったのでコメントを付けています。

scripts/run-mock-api.sh Outdated Show resolved Hide resolved
@mathsuky
Copy link
Collaborator Author

ありがとうございます!指摘いただいた内容を修正したので,最終確認をお願いします:pray:

Copy link
Collaborator

@ZOI-dayo ZOI-dayo left a comment

Choose a reason for hiding this comment

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

ありがとうございます!! 良さそうです!

@mathsuky mathsuky merged commit 8108990 into main Nov 25, 2024
4 checks passed
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