Skip to content
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

Merged
merged 8 commits into from
Jan 12, 2022

Conversation

youngkwon02
Copy link
Member

✅ Default Checklist

  • check branch

  • set Labels

  • set Reviewers


📕 Task

  • 서비스 알림 테스트 데이터 생성

  • 서비스 알림 조회 API

  • 서비스 알림 조회 with Pagination

@youngkwon02 youngkwon02 added 🦋 영권 Enhancement ✨ New feature or request GET 🧡 HTTP method : GET labels Jan 11, 2022
@youngkwon02 youngkwon02 self-assigned this Jan 11, 2022
@youngkwon02 youngkwon02 changed the title [GET] /notice/service/read 서비스 알림 조회 [GET] /notice/service/read?lastid=&size= 서비스 알림 조회 Jan 11, 2022
@youngkwon02 youngkwon02 changed the title [GET] /notice/service/read?lastid=&size= 서비스 알림 조회 [GET] /notice/service?lastid=&size= 서비스 알림 조회 Jan 11, 2022
Comment on lines 27 to 37
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);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

for문대신 .map 을 사용해도 좋을 것 같아 보여요!

Copy link
Member Author

Choose a reason for hiding this comment

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

@xxeol2
map 사용해서 리팩터링 완료했는데, 잘 적용한건지 모르겠네요~!
감사합니당 :)
혹시 더 좋은 사용법 있으면 공유 부탁드려요!!! 😉

Copy link
Contributor

@junghyun-jacky junghyun-jacky left a comment

Choose a reason for hiding this comment

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

good!

@youngkwon02 youngkwon02 merged commit a2ffe6a into develop Jan 12, 2022
@youngkwon02 youngkwon02 deleted the feature/#21 branch January 12, 2022 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement ✨ New feature or request GET 🧡 HTTP method : GET 🦋 영권
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants