Skip to content

feat: add endpoint for fetching user-created notes#293

Closed
Dobrunia wants to merge 1 commit into
mainfrom
feat/my-notes-endpoint
Closed

feat: add endpoint for fetching user-created notes#293
Dobrunia wants to merge 1 commit into
mainfrom
feat/my-notes-endpoint

Conversation

@Dobrunia

@Dobrunia Dobrunia commented Nov 8, 2025

Copy link
Copy Markdown
Member

Summary

This pull request introduces a new /my endpoint that allows authenticated users to retrieve a paginated list of their own notes.

@Dobrunia Dobrunia changed the title feat: add my notes endpoint feat: add endpoint for fetching user-created notes Nov 8, 2025

@neSpecc neSpecc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

reuse existing code

Comment thread src/domain/service/note.ts Outdated
Comment thread src/presentation/http/router/noteList.ts
@Dobrunia Dobrunia force-pushed the feat/my-notes-endpoint branch from 9ddb319 to f348312 Compare November 19, 2025 18:13
@github-actions

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 85.29% (🎯 80%)
⬇️ -0.46%
8873 / 10403
🟢 Statements 85.29% (🎯 80%)
⬇️ -0.46%
8873 / 10403
🔴 Functions 78.67% (🎯 80%)
⬇️ -0.69%
273 / 347
🟢 Branches 84.58% (🎯 80%)
🟰 ±0%
439 / 519
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/domain/service/note.ts 94.08%
⬇️ -1.71%
82.35%
🟰 ±0%
95.45%
⬇️ -4.55%
94.08%
⬇️ -1.71%
100-101, 127-128, 139-140, 168-169, 190-191, 248-257, 296-297, 385-386, 402-403, 444-445, 494-495, 536-537
src/presentation/http/router/noteList.ts 89.13%
⬇️ -10.87%
100%
🟰 ±0%
100%
🟰 ±0%
89.13%
⬇️ -10.87%
118-132
src/repository/note.repository.ts 98.21%
⬇️ -1.79%
100%
🟰 ±0%
91.66%
⬇️ -8.34%
98.21%
⬇️ -1.79%
92-93
src/repository/storage/postgres/orm/sequelize/note.ts 88.71%
⬇️ -8.75%
77.27%
🟰 ±0%
92.85%
⬇️ -7.15%
88.71%
⬇️ -8.75%
191-192, 235-236, 239-240, 296-335, 344-345, 437-438
Generated in workflow #902 for commit f348312 by the Vitest Coverage Report Action

@Dobrunia Dobrunia closed this Nov 19, 2025
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