-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[GET] /notice/service?lastid=&size= 서비스 알림 조회 #22
Conversation
for (let i = 0; i < services.length; i++) { | ||
const service = services[i]; | ||
const notice = { | ||
noticeId: service.notificationId, | ||
noticeTitle: service.title, | ||
noticeImg: service.thumbnail, | ||
noticeContent: service.content, | ||
createdAt: service.createdAt.toISOString().split('T')[0].replace(/-/g, '/'), | ||
}; | ||
notices.push(notice); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for문대신 .map 을 사용해도 좋을 것 같아 보여요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@xxeol2
map 사용해서 리팩터링 완료했는데, 잘 적용한건지 모르겠네요~!
감사합니당 :)
혹시 더 좋은 사용법 있으면 공유 부탁드려요!!! 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good!
✅ Default Checklist
check branch
set Labels
set Reviewers
📕 Task
서비스 알림 테스트 데이터 생성
서비스 알림 조회 API
서비스 알림 조회 with Pagination