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

物材と生物の物質生物の統合対応 #751

Merged
merged 11 commits into from
May 29, 2024

Conversation

Kubosaka
Copy link
Collaborator

@Kubosaka Kubosaka commented May 20, 2024

resolve #750

概要

  • フロントエンドで課程の定数を使っていたため、dbの値を使うように修正した
  • 物材と生物の物質生物の統合を下記のクエリで対応する
    • 今はdepartmentsのid=2は生物機能、id=4は物質材料である
    • このレコードを生物機能→物質生物に変更、id=4に紐づいているteachersを2に変更
    • departmentsのid=4は不要なため削除する
  • 募金の編集をすると登録されている委員が一番上になるバグがまだあったため、追加で修正
// 課程の名前変更
UPDATE departments SET name = "物質生物" WHERE id = 2;
// teachersに紐づいている過程のidを4→2に統一
UPDATE teachers SET department_id = 2 WHERE department_id = 4;
// 不要になったid4の課程の削除
DELETE FROM departments WHERE id = 4;

上記を本番環境のDB上で実行する、
開発環境で上から順番に実行してみて下さい

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

テスト項目

  • 教員一覧ページが表示される
  • 教員一覧で教員の登録、編集ができる
  • 募金一覧ページが表示される
  • 募金一覧で募金の登録・編集ができる
  • dbのdepartmentsを修正した時、フロントでも反映される確認する
  • 募金の編集の際にユーザーが登録されているものから変更されていないか確認する

備考

@github-actions github-actions bot added the bug Something isn't working label May 20, 2024
@Kubosaka Kubosaka requested review from TkymHrt and hikahana May 20, 2024 09:13
@Kubosaka Kubosaka self-assigned this May 20, 2024
@Kubosaka Kubosaka changed the title Fix/kubosaka/750 fix departments data 物材と生物の物質生物の統合対応 May 20, 2024
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

consoleの消し忘れだけ修正お願いします

@@ -29,6 +29,8 @@ export default function EditModal(props: ModalProps) {
receivedAt: props.fundInformation.receivedAt,
});

console.log(formData.userID);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
console.log(formData.userID);

消し忘れだと思います

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

修正しました

@Kubosaka Kubosaka requested a review from hikahana May 27, 2024 02:49
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

@Kubosaka Kubosaka merged commit 31f8c19 into develop May 29, 2024
2 checks passed
@Kubosaka Kubosaka deleted the fix/kubosaka/750-fix-departments-data branch May 29, 2024 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

募金の物材・生物を物質生物へ変更する
2 participants