-
Notifications
You must be signed in to change notification settings - Fork 1
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
「現段階の応募人数」をクラス別に表示 #224
base: develop
Are you sure you want to change the base?
「現段階の応募人数」をクラス別に表示 #224
Conversation
これなんで落ちてるの? |
今回の創作展ではなし |
Code Climate has analyzed commit 09c45f2 and detected 0 issues on this pull request. View more on Code Climate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
一箇所気になったけどそのほかはいいと思います
@@ -138,6 +138,27 @@ def test_get_specific_lottery(client): | |||
assert resp.get_json() == lottery | |||
|
|||
|
|||
def test_lottery_application_num(client): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
このテストは実質schemaのテストだからtest/schemas.py
に入れたほうがいいと思う
委員会「やらないで」だそうです。 |
変更内容
修正後の挙動:
/lotteries/{id} [GET]
のデータにapplication_num
を追加影響範囲
フロントエンドの対応を待ちましょう