📝 상호 평가 - 6조 FE

[honeyyyoon]

안녕하세요! 7조 상호 평가를 담당하게 된 6조 FE 윤시헌, 인소윤, 최서현입니다! 당근에 경매라는 기능을 하여 구현하는 것이 참 매력적이라고 생각합니다😊
앞으로의 페이지 개선에 도움이 될 만한 내용을 정리해보았습니다:)

협업 관련

• Pull Request Template을 설정하게 이에 따라 성실하게 활용하신 점이 매우 인상적이었습니다!
• 두 분이 FE로서 많은 페이지 및 컴포넌트를 구현하신 노고가 잘 보였습니다.-

코드 관련

폴더 구조 개선 제안

• 현재 많은 페이지, CSS, assets 파일이 있는데, 이를 용도 및 페이지별로 폴더화하여 정리하면 더 효율적인 관리가 가능할 것 같습니다.
• 특히 assets의 경우 export 전용 파일을 만들어 비구조화 할당을 활용하면, 더욱 깔끔하게 요소들을 불러올 수 있을 것 같습니다.

export { default as hashtagIcon } from '../assets/hashtag-gray.svg';
export { default as pictureIcon } from '../assets/picture-gray.svg';
export { default as quitIcon } from '../assets/quitcross.svg';
export { default as rightArrow } from '../assets/rightarrow_black.svg';

import { hashtagIcon, pictureIcon, quitIcon, rightArrow } from '@/constants';

이런 방식으로 정리하면 각 컴포넌트에서 반복되는 상수들과 자주 사용되는 아이콘들을 한 곳에서 관리할 수 있어서 유지보수가 더 편해질 것 같습니다!

• public과 assets 폴더의 리소스 용도를 명확히 구분하면 좋을 것 같습니다.

UI/UX 개선 사항

사용자 피드백 강화

• 로그인/회원가입 시 성공/실패에 대한 alert를 추가하면 좋을 것 같습니다.
• item/:id에서는 없는 아이디/에러 시 로딩 화면이 뜨고 에러 alert 가 뜨지 않는 현상, itemedit/:id에서는 없는 아이디/에러 시 /itemedit/:id 페이지 UI 가 그대로 보이는 현상이 있어 적절한 에러 처리 및 alert 설정이 필요할 것 같습니다.
• /login, /location, /itempost, /register 폼에서 입력 형태에 따른 hover 효과(커서 포인터, 색상변화, disabled 상태 표시)를 추가하면 좋을 것 같습니다.

레이아웃 개선

• 검색 환경에서도 하단바가 있으면 좋을 것 같습니다.
• auctions 페이지의 가로 스크롤을 제거하면 좋을 것 같습니다. (width: 100% 또는 overflow-x: hidden 으로 적용하면 좋을 것 같습니다!)
• 기존 당근에서는 프로필 사진을 등록하지 않으면 기본 프로필 아이콘이 뜨는 것 같습니다. 회원가입 후 필수로 프로필 사진을 등록하는 과정이 없고, 당근은 유저들이 소통하는 서비스인 만큼 그냥 회색 이미지 말고 간단한 아이콘이 있으면 좋을 것 같습니다.

기능 개선 사항

버그 수정

• 소셜로그인 시 발생하는 504 Timeout ERROR 해결이 필요합니다.
• 채팅에서 첫 메시지가 중복 전송되는 문제가 있습니다.
• 프로필 사진 업데이트 후 채팅창에 반영되지 않는 문제가 있습니다.

기능 추가 제안

• 중간 총회 때 보면서 ‘경매’라는 요소를 추가한 것이 인상적이었습니다! 다만 현재 경매를 잘 이용할 수 없는 것 같아 아쉬웠습니다. 기말 총회 때는 경매 부분이 구체적으로 구현되면 좋을 것 같습니다!
• 기존 당근에서는 홈 화면에서 지역을 수정할 수 있는데, 이 기능이 추가되어도 좋을 것 같습니다. 지금은 지역 설정을 프로필 수정 페이지에서 수정할 수 있는 것 같은데, 지역 설정을 찾는데 조금 헤멜 수 있는 부분인 것 같습니다!
• 당근이라는 앱 특성상 상품 등록 시 최소 1장 이상의 사진 등록을 필수로 하면 좋을 것 같습니다.

보안 개선

• ItemPage에서 ItemEditPage로 navigate 시 현재 제품 id만 체크하고 있는데, 작성자 본인 확인을 위한 토큰/id 체크 로직 추가가 필요합니다.
• BottomNavBar 접근 권한 관리가 필요합니다:
  • 로그아웃 상태: /chat, /mypage 접근 시 메인 페이지로 리디렉션
  • /main, /auction, /community: 로그인 여부와 관계없이 접근 가능

---

이상으로 피드백을 마치며, 앞으로의 개발에 도움이 되길 바랍니다. 지금까지 수고 많으셨고 남은 일주일도 진심으로 응원합니다! 🎉