-
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
[RN-85] 맛집 리스트 및 랭킹 페이지 구현 #516
Conversation
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.
고생하셨습니다 👍 변경사항이 많지 않으니 빠르게 수정 후 develop에 squash and merge한 다음 작업해주시면 감사하겠습니다~
const locationList = ['전체', '정문', '후문']; | ||
const foodCategoryList = [ | ||
'전체', | ||
'한식', | ||
'일식', | ||
'중식', | ||
'양식', | ||
'분식', | ||
'간편식', | ||
'기타', | ||
]; |
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.
바뀔 수 있는 상수는 해당 도메인 폴더에 constant 폴더를 생성해서 따로 관리하는 것을 추천합니다
const navigation = useNavigation<RootNavigationProps>(); | ||
const [bottomSheetItem, setBottomSheetItem] = | ||
useState<RestaurantItemType | null>(); | ||
const [openBottomSheet, closeBottomSheet, BottomSheet] = |
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.
closeBottonSheet()
함수를 사용하지 않는다면 _
underscore로 지정하시면 알아보기 좋을 것 같습니다.
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.
underscore 사용하면 eslint 위반되는데 그대로 진행시키면 되나요..?
export type RestaurantItemType = { | ||
name: string; | ||
location: string; | ||
restaurantType: string; | ||
like: boolean; | ||
likesCount: number; | ||
mapLink: string; | ||
}; |
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.
해당 TF에서 전체적으로 사용하는 타입이라, 도메인애 types 폴더를 만들어서 따로 관리하면 좋을 것 같습니다.
merge origin/RN-85 to RN-85
Key Changes
To Reviewers
현재 RestaurantScreen에서 맛집 리스트 요소 press 시 하단 모달을 띄우기 위해 state(bottomSheetItem)를 사용하고 있습니다.
state로 bottomSheet에 선택된 요소 정보를 전달하는 방법 이외에 다른 방법이 있을까요?
BorderSelect 컴포넌트에서 boxShadow 스타일 적용이 안되는데 이유를 모르겠습니다!
도서관 순위 페이지에서도 Select 컴포넌트에서 적용이 안된 것 같습니다.