Skip to content

ドキュメント追加とDBスキーマの改善#47

Merged
newt239 merged 17 commits intomainfrom
chore/improve-architecture
Apr 11, 2025
Merged

ドキュメント追加とDBスキーマの改善#47
newt239 merged 17 commits intomainfrom
chore/improve-architecture

Conversation

@newt239
Copy link
Member

@newt239 newt239 commented Apr 6, 2025

Caution

Prismaのマイグレーションファイルを作り直したので、Dockerコンテナおよびイメージ・ボリュームを削除したうえで、DevContainerの再構築をお願いします。

やったこと

  • ERD.md, ADR.md, CONTRIBUTING.mdの追加
  • データベースのスキーマ変更とマイグレーションファイルのリセット
    • UserRoleテーブルのpermissions: jsonからstringに変更
    • Workspaceテーブルのid: uuid制約を外し、一意なIDを設定できるように(URLのサブドメインに使う可能性もあるため)
    • Channelテーブル, Threadテーブルのdescription: nullableに変更
    • Channelテーブル: nameカラムを削除
    • Assetテーブル: 代替テキストを設定するaltカラムを追加
    • 型がDateTimeのカラムを、Auth.js関連のテーブルを除き〇〇Atという形に統一
  • 依存関係の更新
    • DependabotでNext.jsの脆弱性に関する警告が出ていたのでアップグレードしたのと、合わせてその他のライブラリもアップグレードしました
    • pnpmの仕様変更によりローカルではなくコンテナ上でpnpmを実行する必要があるかも
  • publicディレクトリの未使用svgファイルを削除

@newt239 newt239 self-assigned this Apr 6, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 13 out of 20 changed files in this pull request and generated 1 comment.

Files not reviewed (7)
  • package.json: Language not supported
  • prisma/migrations/20250104110708_fix_leave_date_nullable/migration.sql: Language not supported
  • prisma/migrations/20250204171719_auth_setup/migration.sql: Language not supported
  • prisma/migrations/20250225110302_add_message_links_table/migration.sql: Language not supported
  • prisma/migrations/20250226095622_add_updated_at/migration.sql: Language not supported
  • prisma/migrations/20250406094624_initial_schema/migration.sql: Language not supported
  • prisma/schema.prisma: Language not supported

Copy link
Contributor

@saka-naname saka-naname left a comment

Choose a reason for hiding this comment

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

LGTMeow

よさそう

@newt239 newt239 merged commit d782450 into main Apr 11, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants