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

expenseを登録・削除・編集できるようにする+その他バグ修正 #548

Merged
merged 9 commits into from
Apr 23, 2023

Conversation

imaimai17468
Copy link
Collaborator

@imaimai17468 imaimai17468 commented Apr 23, 2023

対応Issue

resolve #544
resolve #545
resolve #546

概要

  • 支出を追加・削除・編集をできるようにする
  • 協賛活動で編集・削除すると詳細モーダルも同時に開くのを修正する
  • チェックボックスと編集・削除のdisabledを連動させる

テスト項目

test3でログインする

  • 支出を追加・編集・削除できるか
  • 協賛活動で編集・削除すると詳細モーダルも同時に開くのを修正する
  • 購入申請・報告で、チェックボックスのチェックをつけ外しすると、それに沿って編集ボタンと削除ボタンのdisabledが切り替わるか

備考

@imaimai17468 imaimai17468 added bug Something isn't working enhancement New feature or request labels Apr 23, 2023
@imaimai17468 imaimai17468 self-assigned this Apr 23, 2023
@Kubosaka
Copy link
Collaborator

動作問題なしです。
支出の作成、更新、削除するとページリロードされるけど、今のままやと予算一覧が表示されちゃうけどこれ治せそう?

@imaimai17468
Copy link
Collaborator Author

@Kubosaka

予算/支出はタブで切り替えてるんだけど、リロード前のタブの値を保存するのは少し面倒だから、別issueで切ってこのPRは一旦このままで行きたい!

@Kubosaka
Copy link
Collaborator

それなら問題なしです!

Copy link
Collaborator

@Kubosaka Kubosaka 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 4617b82 into develop Apr 23, 2023
@imaimai17468 imaimai17468 deleted the feature/imaimai/expense-edit branch April 23, 2023 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
2 participants