fix #37 진수원 css selector 문제 - Githubissues

su-mmer / JBNU_OSS_PROJECT

오픈소스 소프트웨어 슬랙봇 개발 프로젝트(이제 CI를 곁들인)

1 stars 1 forks source link

fix #37 진수원 css selector 문제 #38

Closed su-mmer closed 1 year ago

su-mmer commented 1 year ago

[x] PR 전 테스트를 통과했습니까?

개요(#37)

lunch.js 파일에서 css selector에 필터링을 추가하고 menu 배열에서 인덱스를 일일이 지정하는 하드코딩 부분을 수정했습니다.

변경 또는 추가 사항

lunch.js 파일 변경
css selector에 has로 필터링 추가, span과 font를 가진 selector는 모두 elements에 저장됩니다.
element에서 모든 요소를 불러와 res에 저장하고 그 중 인덱스의 번호를 일일이 지정하여 menu 배열을 생성하는 과정을 없앴습니다.
코드 동작 시 추가적으로 팀원들이 해야할 일이 있는지?
- 없습니다.

리뷰어에게 할 말

리뷰어가 집중적으로 보아야할 부분은?
- selector에 필터링 추가
- 24번줄에서보면 menu에 메뉴를 이중배열로 전부 집어넣는 대신 필요한 만큼만 res에서 꺼내오도록 지정했습니다.
리뷰어가 신경쓰지 않아도 될 부분은? e.g. 줄바꿈
- selector부분 외의 예외처리 및 평점 코드는 신경쓰지 않아도 됩니다.

스크린샷(필요한 경우)

다른 팀원들이 동일한 작동을 하는지 확인하기 위해 스크린샷을 첨부해주세요.
res 출력시 다음과 같이 출력됩니다.
오늘의 메뉴를 추출해 답변하는 슬랙봇 화면입니다.