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

年(年度)指定の追加 #557

Merged
merged 7 commits into from
Apr 29, 2023
Merged

年(年度)指定の追加 #557

merged 7 commits into from
Apr 29, 2023

Conversation

KazumaSun
Copy link
Collaborator

@KazumaSun KazumaSun commented Apr 25, 2023

対応Issue

resolve #503

概要

セレクトボックスで指定された年(年度)の情報のみを表示させるようにした.
予算ページ,購入申請ページ,購入報告ページでは,合計金額にも反映するようにした.

画面スクリーンショット等

  • URL
    スクリーンショット
    例(「学内募金」)
    finansu5

テスト項目

「予算」,「学内募金」,「購入申請」,「購入報告」,「協賛活動」,「協賛企業」のページで年(年度)指定で表示内容が切り替えられるか
「予算」,「学内募金」,「購入申請」,「購入報告」の合計金額が年(年度)ごとに変更されるか確認.
なお,「学内募金」では,test3@test.com(財務局長権限)によるチェックが完了しているものの合計金額とした.

備考

@KazumaSun KazumaSun self-assigned this Apr 25, 2023
@imaimai17468 imaimai17468 added the enhancement New feature or request label Apr 27, 2023
Copy link
Collaborator

@imaimai17468 imaimai17468 left a comment

Choose a reason for hiding this comment

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

実装ありがとう!
方針はいい感じで動作も問題なし!

  • BudgetはYear型を使っているのでこのままで大丈夫
  • それ以外は、purchaseOrderに対するレビューを参考に全体的に修正して欲しい!
  • あとコンフリクトも解消してほしい

view/next-project/src/pages/purchaseorders/index.tsx Outdated Show resolved Hide resolved
view/next-project/src/pages/purchaseorders/index.tsx Outdated Show resolved Hide resolved
view/next-project/src/pages/purchaseorders/index.tsx Outdated Show resolved Hide resolved
view/next-project/src/pages/purchaseorders/index.tsx Outdated Show resolved Hide resolved
view/next-project/src/pages/purchaseorders/index.tsx Outdated Show resolved Hide resolved
@KazumaSun KazumaSun added enhancement New feature or request and removed enhancement New feature or request labels Apr 28, 2023
Copy link
Collaborator

@imaimai17468 imaimai17468 left a comment

Choose a reason for hiding this comment

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

lgtm!

@imaimai17468 imaimai17468 merged commit 483db7b into develop Apr 29, 2023
@imaimai17468 imaimai17468 deleted the feature/sato/select-years branch April 29, 2023 04:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

年を指定するselect要素で切り替えられるようにする
2 participants