[Feature]: 제휴/행사 검색, 좋아요 토스트, 게시글 삭제 에러처리#68
Merged
Conversation
- CouncilAffiliateDetailScreen 헤더에 ⋮ 버튼 추가 - EditPostBottomSheet 연동으로 수정/삭제 메뉴 표시 - 삭제 시 ConfirmModal로 확인 다이얼로그 표시 - ConfirmModal 공용 컴포넌트 신규 생성
- hasNavBar prop으로 위치 동적 계산 - 탭바 있을 때: safeArea + tabBarHeight(49) + 22px - 탭바 없을 때: screenHeight - 736px - tabBarHeight prop으로 커스텀 탭바 높이 지원
- CouncilAffiliateDetailScreen: 삭제 API 에러 처리 및 로딩 상태 추가 - Toast: 하드코딩 fontSize → typography.body4Regular 토큰으로 교체
daeun088
added
♻️ refactor
daeun088
changed the title
MlNTYS
reviewed
Feb 24, 2026
| setIsDeleteModalVisible(false); | ||
| setIsDeleting(true); | ||
| try { | ||
| await deleteCouncilPost(postId, accessToken); |
Contributor
There was a problem hiding this comment.
SHOULD: councilAffiliate.js 쪽 보니까 return쪽에서 error를 throw해주지 않아고 삼켜버려서 아래의 catch가 실행되지 않을 거 같습니다..!
MlNTYS
reviewed
Feb 24, 2026
src/components/common/Toast.js
Outdated
| container: { | ||
| position: 'absolute', | ||
| alignSelf: 'center', | ||
| backgroundColor: '#595F63CC', |
Contributor
There was a problem hiding this comment.
SHOULD: colors.gray[700]에 투명도 80%가 더 좋아 보입니다!
MlNTYS
reviewed
Feb 24, 2026
src/components/common/Toast.js
Outdated
| borderRadius: 50, | ||
| }, | ||
| text: { | ||
| color: 'white', |
Contributor
There was a problem hiding this comment.
SHOULD: 이 부분도 colors.common.white 가 맞아 보여요!
MlNTYS
reviewed
Feb 24, 2026
src/components/common/Toast.js
Outdated
|
|
||
| const bottomPosition = hasNavBar | ||
| ? insets.bottom + tabBarHeight + 22 | ||
| : screenHeight - 736; |
Contributor
There was a problem hiding this comment.
SHOULD: 상수보다는 여기도 뭔가 동적으로 계산할 수 있으면 좋을 거 같아요.
MlNTYS
reviewed
Feb 24, 2026
| const [isEmpty, setIsEmpty] = useState(true); | ||
| const [recommendData, setRecommendData] = useState([]); | ||
| const [imagesLoaded, setImagesLoaded] = useState({}); // 각 이미지의 로딩 상태 추적 | ||
| const [toastVisible, setToastVisible] = useState(false); |
Contributor
There was a problem hiding this comment.
SHOULD: 이 부분이 계속 여기저기 반복되는 거 같은데, 커스텀 훅이나 중간에서 관리하는 방법이 있으면 좋을 거 같습니다..!
MlNTYS
requested changes
Feb 24, 2026
Contributor
MlNTYS
left a comment
MlNTYS
left a comment
There was a problem hiding this comment.
수고하셨습니다! councilAffiliate.js 부분만 잘 봐주시면 좋을 거 같습니다! 수고하셨어요!
- deleteCouncilPost: error catch 후 re-throw 추가 - Toast: backgroundColor/color 하드코딩 → colors 토큰으로 교체 - Toast: bottomPosition → insets.bottom + 22 동적 처리 - colors: overlay.toast 토큰 추가 - useToast 커스텀 훅 추출 - 5개 스크린에 useToast 적용
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 관련 이슈
close #67
✨ 변경 사항
제휴/행사 목록 검색 기능, 좋아요·게시글 등록 토스트 알림, 게시글 삭제 에러 처리, 공용 Toast/ConfirmModal 컴포넌트 추가
🧩 세부 내용
AffiliationMainScreen: 검색 모드 UI 추가 (검색바 ↔ 칩 전환, 클라이언트 사이드 필터링)AffiliationMainScreen검색 Row 스타일 통일 (패딩, 아이콘 크기/위치, 배경색)AffiliationMainScreen/AffiliationDetailScreen/AffiliationLikedScreen: 좋아요 추가/취소 토스트 연결SelectAffiliationLogoScreen/WriteEventPostScreen: 게시글 등록 토스트 연결 (react-native-toast-message→ 공용Toast컴포넌트로 교체)CouncilAffiliateDetailScreen: 게시글 삭제 API 에러 처리 및 로딩 상태(isDeleting) 추가Toast.js컴포넌트 추가 (bottom 포지셔닝, 애니메이션,hasNavBar옵션)ConfirmModal.js컴포넌트 추가Toast텍스트 스타일 하드코딩fontSize→typography.body4Regular토큰으로 교체📸 스크린샷 (선택)
💬 기타 참고사항
CustomToast별도 유지 — 신규Toast.js는 화면 하단 포지션으로 용도 구분