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

[feat] 아이템 정보 파싱 api 구현 #185

Merged
merged 1 commit into from
Apr 16, 2022
Merged

Conversation

hyejungg
Copy link
Member

@hyejungg hyejungg commented Apr 8, 2022

What is this PR? 🔍

아이템 정보를 파싱하는 api를 구현하였습니다.

Key Changes 🔑

  1. parser.js의 형태는 기존의 안드에서 파싱하는 방법과 유사합니다. (코드 구조가 좋아서 손민수.)

To Reviewers 📢

  • 라이브러리가 추가되었습니다. 오픈소스 라이브러리 재업로드 필요!
  • lib/parser.js로 뺐는데 모듈위치가 여기가 적합한지 잘 모르겠네요. 어케 생각하심이?
  • 테스트 후 확인 부탁드립니다! (eql스토어는 production 모드도 안되네요)

@hyejungg hyejungg added the feat 새 기능 추가 label Apr 8, 2022
@hyejungg hyejungg requested a review from youngjinc April 8, 2022 16:30
@hyejungg hyejungg self-assigned this Apr 8, 2022
@hyejungg hyejungg added this to the Item 구현 기능 milestone Apr 8, 2022
@hyejungg hyejungg linked an issue Apr 8, 2022 that may be closed by this pull request
2 tasks
@youngjinc
Copy link
Member

eql스토어는 production 모드도 안되네요..

꽤나 충격적

}
break;
case 'price':
case 'Price':
Copy link
Member

Choose a reason for hiding this comment

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

이건 뭘까요..? case 'price': 중복됐습니다..?

Copy link
Member Author

Choose a reason for hiding this comment

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

영진이코드 보니까 정규식으로 price / Price 둘 다 받길래 두 개 써봤습니다 ㅎㅎ..

Copy link
Member

Choose a reason for hiding this comment

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

아 대문자와 소문자 차이가 있었군요..

@hyejungg hyejungg merged commit 4d74ff9 into dev Apr 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 새 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

아이템 정보 파싱 api 구현
2 participants