Skip to content
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

Merged
merged 26 commits into from
Nov 14, 2024

Conversation

leedongyull
Copy link
Collaborator

@leedongyull leedongyull commented Nov 13, 2024

📝 PR 개요

AddChannel 페이지 UI 구현


🔍 변경 사항

UI 구현
Figma에 맞게 디자인 수정
tailwind 색상 추가
Layout 적용


✅ 체크리스트 (Checklist)

  • 코드가 빌드 오류 없이 잘 작동하는지 확인
  • 테스트가 통과하는지 확인
  • 스타일 가이드와 일관성을 유지했는지 확인
  • 관련 문서가 업데이트되었는지 확인 (선택 사항)
  • 리뷰어가 이해할 수 있도록 주석이나 설명을 추가했는지 확인

🔄 관련 이슈 (Linked Issues)

#106 작업


📷 스크린샷 및 동영상 (선택 사항)

ViteReactTS-Chrome2024-11-1408-39-33-ezgif com-video-to-gif-converter

화면 GIF

사용자 삭제 없어짐

사용자 5명 이상일때, 삭제 버튼 없어짐

삭제 테스트

mockData 이용해서 테스트 3번 사용자 삭제 시 4,5번 사용자가 3,4번으로 당겨짐
swagger 보니 사용자 구분은 이름으로 하고 id는 사용 안하는 것 같아서 인덱스 대용으로 구현했습니다!


참고사항

  • 아직 경로 설정 컴포넌트 구현은 못했습니다 (지도 페이지와 연동해서 마커의 정보를 받아 와야함 + 받아 오면 디자인도 변경 돼야함)
  • 따라서 아직 하단 Footer 버튼의 상태 변화도 구현하지 않았습니다!
  • 추후 마커 설정 기능이 완료되면 추가할 예정입니다
  • 상단 Header의 경우도 아직 뒤로가기 버튼이 구현되지 않아 없는 상태입니다
  • 영상은 폰트가 적용되어 있지 않아 어색해 보일 수 있습니다!

@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 13, 2024
Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

5명 추가까지 되면 사용자 추가 버튼 안 보이게 해주세요!
그리고 사용자2부터는 오른쪽에 X 버튼 넣고 사용자 없앨 수 있게 해주세요!
폰트 설정안 되어 있는거 확인 부탁드립니다!

frontend/src/App.tsx Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@juwon5272 juwon5272 added 수정 요청 특정 이유로 PR을 approve 할 수 없어 반려한 상태 (리뷰어 2명 중 PR 반려 한 사람이 상태 변경) and removed 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) labels Nov 14, 2024
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 14, 2024
@github-actions github-actions bot requested a review from juwon5272 November 14, 2024 04:52
@leedongyull leedongyull removed the 수정 요청 특정 이유로 PR을 approve 할 수 없어 반려한 상태 (리뷰어 2명 중 PR 반려 한 사람이 상태 변경) label Nov 14, 2024
Copy link
Member

@happyhyep happyhyep left a 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를 추가하게 될 것 같습니다! 아니면 이름이 중복되지 않게 하는 로직 정도 추가하게 될 것 같아요! 이 부분은 조금 더 생각해본 뒤 다시 말씀드릴게요!! 우선은 지금 방식으로 해주셔도 될 것 같습니다!

frontend/src/pages/AddChannel.tsx Outdated Show resolved Hide resolved
frontend/src/pages/AddChannel.tsx Show resolved Hide resolved
frontend/src/pages/AddChannel.tsx Outdated Show resolved Hide resolved
frontend/src/pages/AddChannel.tsx Outdated Show resolved Hide resolved
@juwon5272
Copy link
Collaborator

dev브랜치에서 합칠때 BE 먼저 합치고 FE 합칠거 같은데, 그렇게 되면 기존 BE의 package.json이 사라지거나 충돌날거 같아 삭제가 필요할거 같습니다! 프론트엔드나 공통의 package.json도 가능하시다면 부탁드려요

backend/package.json Outdated Show resolved Hide resolved
@leedongyull leedongyull added 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) and removed 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) labels Nov 14, 2024
@github-actions github-actions bot requested a review from happyhyep November 14, 2024 09:04
@leedongyull leedongyull requested a review from effozen November 14, 2024 10:06
@leedongyull leedongyull merged commit f176c60 into frontend Nov 14, 2024
1 check failed
@leedongyull leedongyull deleted the feature/fe/#106-AddChannelUI branch November 17, 2024 03:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants