Skip to content

Conversation

@Dobrunia
Copy link
Member

@Dobrunia Dobrunia commented Nov 13, 2025

Summary

Renamed getNoteListByUserId method to getRecentNotesByUserId across all layers (storage, repository, service) to better reflect that this method returns notes ordered by time of last visit, not just all notes created by a user.

Changes

  • Storage layer: Renamed getNoteListByUserIdgetRecentNotesByUserId in NoteSequelizeStorage
  • Repository layer: Renamed getNoteListByUserIdgetRecentNotesByUserId in NoteRepository
  • Service layer: Renamed getNoteListByUserIdgetRecentNotesByUserId in NoteService
  • Router: Updated method call in noteList.ts router
  • Documentation: Updated JSDoc comments to clarify that the method returns notes ordered by last visit time

@github-actions
Copy link

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 85.75% (🎯 80%)
🟰 ±0%
8808 / 10271
🟢 Statements 85.75% (🎯 80%)
🟰 ±0%
8808 / 10271
🔴 Functions 79.36% (🎯 80%)
🟰 ±0%
273 / 344
🟢 Branches 84.58% (🎯 80%)
🟰 ±0%
439 / 519
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/domain/service/note.ts 95.79%
🟰 ±0%
82.35%
🟰 ±0%
100%
🟰 ±0%
95.79%
🟰 ±0%
100-101, 127-128, 139-140, 168-169, 190-191, 278-279, 367-368, 384-385, 426-427, 476-477, 518-519
src/presentation/http/router/noteList.ts 100%
🟰 ±0%
100%
🟰 ±0%
100%
🟰 ±0%
100%
🟰 ±0%
src/repository/note.repository.ts 100%
🟰 ±0%
100%
🟰 ±0%
100%
🟰 ±0%
100%
🟰 ±0%
src/repository/storage/postgres/orm/sequelize/note.ts 97.46%
🟰 ±0%
77.27%
🟰 ±0%
100%
🟰 ±0%
97.46%
🟰 ±0%
191-192, 235-236, 239-240, 295-296, 388-389
Generated in workflow #900 for commit 4607b74 by the Vitest Coverage Report Action

@Dobrunia Dobrunia merged commit 9f2f501 into main Nov 15, 2025
6 checks passed
@Dobrunia Dobrunia deleted the refactor/rename-notes-endpoint branch November 15, 2025 18:30
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.

4 participants