Skip to content

Add place notes field to photos - #93

Draft
koo5 with Copilot wants to merge 3 commits into
masterfrom
copilot/koo5-hillview-92-add-notes-column
Draft

Add place notes field to photos#93
koo5 with Copilot wants to merge 3 commits into
masterfrom
copilot/koo5-hillview-92-add-notes-column

Conversation

Copilot AI commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

This adds a dedicated notes field to photos for place-specific context, separate from the existing photo description. The backend now stores, accepts, and returns notes explicitly, while keeping room for future annotation-style editing/licensing mechanics.

  • Schema

    • Add nullable notes column to photos
    • Include Alembic migration for the new field
  • Upload + persistence

    • Accept notes in the upload authorization request
    • Persist notes onto the Photo model alongside existing metadata
  • API surface

    • Return notes from photo-facing endpoints that already expose photo metadata
    • Omit empty notes values from responses for consistency with existing optional fields
  • Content selection logic

    • Treat notes as meaningful photo metadata in places that decide whether a photo is “contentful” enough for sitemap/place flows
  • Focused test coverage

    • Add unit coverage for request parsing and response serialization of notes
class UploadAuthorizationRequest(BaseModel):
    title: Optional[str] = None
    description: Optional[str] = None
    notes: Optional[str] = None
    keywords: Optional[list[str]] = None

Copilot AI changed the title [WIP] Add notes column for each photo Add place notes field to photos Jun 25, 2026
Copilot finished work on behalf of koo5 June 25, 2026 20:55
Copilot AI requested a review from koo5 June 25, 2026 20:55
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