-
Notifications
You must be signed in to change notification settings - Fork 2
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
[FE][Feat] #106 - AddChannel 페이지 구현 #161
Conversation
…into feature/fe/#106-AddChannelUI
…com/boostcampwm-2024/web28-DDara into feature/fe/#106-AddChannelUI
…stcampwm-2024/web28-DDara into feature/fe/#106-AddChannelUI
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.
5명 추가까지 되면 사용자 추가 버튼
안 보이게 해주세요!
그리고 사용자2부터는 오른쪽에 X 버튼 넣고 사용자 없앨 수 있게 해주세요!
폰트 설정안 되어 있는거 확인 부탁드립니다!
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.
수정 반영해주신 거 확인했습니다!
guest name으로 구분핮ㄴ다고 말씀해주셔서 다시 생각해보니, index로 구분하는게 필요하다는 생각이 들어 api 수정해두겠습니다! marker_style에 user_index를 추가하게 될 것 같습니다! 아니면 이름이 중복되지 않게 하는 로직 정도 추가하게 될 것 같아요! 이 부분은 조금 더 생각해본 뒤 다시 말씀드릴게요!! 우선은 지금 방식으로 해주셔도 될 것 같습니다!
dev브랜치에서 합칠때 BE 먼저 합치고 FE 합칠거 같은데, 그렇게 되면 기존 BE의 package.json이 사라지거나 충돌날거 같아 삭제가 필요할거 같습니다! 프론트엔드나 공통의 package.json도 가능하시다면 부탁드려요 |
📝 PR 개요
🔍 변경 사항
✅ 체크리스트 (Checklist)
🔄 관련 이슈 (Linked Issues)
📷 스크린샷 및 동영상 (선택 사항)
참고사항