-
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
[fix] 年度別に支出の選択欄を選べるように変更 #710
[fix] 年度別に支出の選択欄を選べるように変更 #710
Conversation
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.
支出を最新の年度を選択肢にしてください
console.log(selectedYear); | ||
console.log(props.expenses); | ||
console.log(parseInt(props.selectedYear)); | ||
console.log(props.yearPeriods); |
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.
consoleのログを消してください
年度で支出を取得するAPIを作成して |
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.
APIは問題ないです。
取得したexpensesがnullの場合、登録ボタンを押すとエラー起きるので、対応してほしいです。
例:最新の年度を作成した際に、まだexpenesが未登録の場合など
対応Issue
resolve #708
概要
購入申請ページの新規登録モーダルにて、支出欄を年度別に切り替えられるようにしました
画面スクリーンショット等
URL
スクリーンショット
テスト項目
備考
適当にpropsを投げまくっているためコードよく見てほしいです