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

[fix] 登録されたデータが編集モーダルに反映されるよう修正 #696

Merged
merged 5 commits into from
Mar 22, 2024

Conversation

TkymHrt
Copy link
Collaborator

@TkymHrt TkymHrt commented Mar 5, 2024

対応Issue

resolve #663

概要

Issue内容は、担当者が各局の一番上の人になるバグの解消です。
しかし、教員の所属・教員名・担当者の局の項目でも同様のバグが発生していたため、まとめて修正しました。
修正内容は、各項目の初期値として登録される値を、既に登録されている内容から参照するように変更しました。

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

FireShot Capture 169 - 学内募金一覧 - localhost

FireShot Capture 170 - 学内募金一覧 - localhost

テスト項目

  • FinanSuをローカルで立ち上げ、http://localhost:3000/fund_informations へアクセスできることを確認する。
  • 登録項目の編集ボタンを押し、編集モーダルが開くことを確認する。
  • 各項目を初期値以外のものに設定し、登録ボタンを押す。(スクショ画像を参考にしてください。)
  • 登録できたことを確認して再度、編集ボタンを押し、登録内容が初期化されていないかを確認する。
    (各項目が全て1番目の要素に初期化されていないか。)

備考

24/03/29 更新:モーダルの展開時はpropsの値を表示し、セレクト要素の選択時にUseEffectの再レンダリングで教員名の初期値を設定するようにしました。もっと良い方法あれば教えて下さい。

@TkymHrt TkymHrt self-assigned this Mar 5, 2024
@github-actions github-actions bot added the bug Something isn't working label Mar 5, 2024
@TkymHrt TkymHrt requested a review from hikahana March 8, 2024 04:26
Copy link
Collaborator

@hikahana hikahana left a comment

Choose a reason for hiding this comment

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

動作確認できました。
ちょっと気になったところコメントしてます

Comment on lines 35 to 55
useEffect(() => {
const teacher = props.teachers.find((teacher) => teacher.departmentID === departmentID);
if (teacher && teacher.id) {
setFormData({ ...formData, teacherID: teacher.id });
if (defaultValue == false) {
const selectedTeacher = props.teachers.find(
(teacher) => teacher.id === props.fundInformation.teacherID,
);
if (selectedTeacher?.id) {
setDepartmentID(selectedTeacher.departmentID);
setFormData({ ...formData, teacherID: selectedTeacher.id });
}
setTimeout(() => {
setDefaultValue(true);
});
} else {
const relatedTeachers = props.teachers.filter(
(teacher) => teacher.departmentID === departmentID,
);
if (relatedTeachers) {
setFormData({ ...formData, teacherID: relatedTeachers[0].id || 0 });
}
}
}, [departmentID]);
}, [departmentID, props.teachers]);
Copy link
Collaborator

Choose a reason for hiding this comment

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

defaultValueなくてもいけるのではないでしょうか

  useEffect(() => {
    const selectedTeacher = props.teachers.find(
      (teacher) => teacher.id === props.fundInformation.teacherID,
    );
    if (selectedTeacher?.id) {
      setDepartmentID(selectedTeacher.departmentID);
      setFormData({ ...formData, teacherID: selectedTeacher.id });
    } else {
      const relatedTeachers = props.teachers.filter(
        (teacher) => teacher.departmentID === departmentID,
      );
      if (relatedTeachers) {
        setFormData({ ...formData, teacherID: relatedTeachers[0].id || 0 });
      }
    }
  }, [departmentID, props.teachers]);

Copy link
Collaborator

Choose a reason for hiding this comment

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

PS
いけませんでした。忘れてください

Copy link
Collaborator

@hikahana hikahana left a comment

Choose a reason for hiding this comment

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

LGTM

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.

対応ありがとうございます!
少し、コメントしました!
動作おかしいやこっちのがいいなどあれば気軽に言ってください

@Kubosaka Kubosaka self-requested a review March 22, 2024 07:38
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

@Kubosaka Kubosaka merged commit d07bbea into develop Mar 22, 2024
2 checks passed
@Kubosaka Kubosaka deleted the fix/yama/663-fix-modal-bug branch March 22, 2024 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

募金ページの編集モーダルのバグ修正
3 participants