Skip to content

[Week7 - mjk]#61

Open
openkmj wants to merge 2 commits into
mainfrom
week7-mjk
Open

[Week7 - mjk]#61
openkmj wants to merge 2 commits into
mainfrom
week7-mjk

Conversation

@openkmj

@openkmj openkmj commented Mar 17, 2026

Copy link
Copy Markdown

Summary

  • Task 1: Added DELETE & count endpoints for Notes/ActionItems, Pydantic Field validation (min_length, max_length), Path/Query parameter validation
  • Task 2: Enhanced extract_action_items() with structured metadata output (priority, assignee, deadline detection), added extraction endpoint POST /notes/{id}/extract-actions
  • Task 3: Created Tag model with many-to-many relationship to Note, full CRUD /tags/ endpoints, note-tag association on create/patch
  • Task 4: Expanded test coverage from 3 to 30 tests — pagination boundary cases, sorting (asc/desc/fallback), search+pagination, filter+pagination

Note: The requirement to split tasks into 4 separate PRs has been removed from the assignment. All tasks are submitted in a single PR.

Test plan

  • make test — 30/30 tests passing
  • make format — black + ruff clean
  • Manual verification of new endpoints via FastAPI docs (/docs)

openkmj added 2 commits March 17, 2026 17:31
- Task 1: DELETE/count 엔드포인트 추가, Pydantic Field validation
- Task 2: extract_action_items 고도화 (priority, assignee, deadline 메타데이터)
- Task 3: Tag 모델 + Note-Tag M2M relationship, /tags/ CRUD
- Task 4: 페이지네이션/정렬/필터 테스트 커버리지 개선 (3 → 30 tests)
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.

1 participant