Skip to content

みんなの描いた絵#245

Merged
toyoshin5 merged 2 commits into
mainfrom
feature/expo
Nov 9, 2025
Merged

みんなの描いた絵#245
toyoshin5 merged 2 commits into
mainfrom
feature/expo

Conversation

@toyoshin5

@toyoshin5 toyoshin5 commented Nov 9, 2025

Copy link
Copy Markdown
Collaborator

関連issue

変更概要

みんなの描いた絵を実装

動作確認方法やスクリーンショット

 2025-11-09 9 51 14

Copilot AI review requested due to automatic review settings November 9, 2025 00:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

「みんなの描いた絵」機能の実装により、EXPOで書かれた一筆書きを表示するページが追加されました。この機能では、ユーザーが描いた手書きデータをバックエンドに保存し、フロントエンドで一覧表示できるようになっています。

  • バックエンドに手書きデータを保存・取得するエンドポイントを追加
  • フロントエンドにEXPOページを新規作成し、手書きデータをキャンバスでプレビュー表示
  • 日時フィルタリング機能を実装し、指定日時以降のデータを取得可能に

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
frontend/src/app/expo/page.tsx 手書きデータを一覧表示するEXPOページの新規作成。キャンバスプレビュー機能と日時フィルタリング機能を実装
backend/schemas.py 手書きデータ用のPydanticスキーマ(HandwritingBase, HandwritingCreate, Handwriting)を追加
backend/models.py 手書きデータを保存するHandwritingモデルをデータベースに追加
backend/main.py ルート計算時に手書きデータを保存する処理と、手書きデータを取得するGETエンドポイントを追加

Comment thread frontend/src/app/expo/page.tsx
Comment thread frontend/src/app/expo/page.tsx
Comment thread backend/schemas.py


class HandwritingBase(BaseModel):
drawing_points: list[DisplayPoint]

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

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

DisplayPointが定義されていません。このスキーマを使用する前に、DisplayPointの定義を追加する必要があります。

Copilot uses AI. Check for mistakes.
Comment thread backend/models.py

class Handwriting(Base):
__tablename__ = 'handwritings'
id = Column(UUID, primary_key=True, default=uuid.uuid4)

Copilot AI Nov 9, 2025

Copy link

Choose a reason for hiding this comment

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

uuidがインポートされていません。ファイルの先頭にimport uuidを追加する必要があります。

Copilot uses AI. Check for mistakes.
@toyoshin5 toyoshin5 merged commit 44ba0f0 into main Nov 9, 2025
2 checks 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.

2 participants