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

전남대_12조_픽업셔틀_5주차 #40

Merged
merged 61 commits into from
Oct 11, 2023
Merged

전남대_12조_픽업셔틀_5주차 #40

merged 61 commits into from
Oct 11, 2023

Conversation

rktdnjs
Copy link
Contributor

@rktdnjs rktdnjs commented Oct 8, 2023

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • CSS등 UI 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

전달 사항

안녕하십니까 멘토님! 이번 5주차 개발 진행 사항을 모아 PR 드립니다!
이번주차 멘토링 사전질문 & 개발 내용과 관련된 질문은 멘토링 전까지 하단 링크에 별도로 정리해놓도록 하겠습니다😊

질문 사항을 정리한 링크입니다! (멘토 피드백 일지)

https://www.notion.so/FE-6-cea6d5ba686a490abef3367457208d2d

작업 사항

참고 : 이번 주차 내용을 review에 PR 거는 과정에서 4주차 review 브랜치 Merge전 5주차 내용을 master 브랜치에 Merge 해버려
4주차 review Merge 과정에서 5주차 내용도 섞여 들어가버렸습니다(...) 그래서 부득이하게 이번만 review2 브랜치를 파서
5주차 내용에 대한 내용을 피드백 받고자 하였습니다! 6주차부터는 다시 review 브랜치에 PR 걸도록 하겠습니다😭

5주차 전반적인 개발 사항 정리

  • 메인화면, 로그인, 공고현황, 공고작성, 공고 상세페이지 레이아웃 구체화
  • LoginNav 컴포넌트 구현(→OtherNav 컴포넌트로 통일)
  • 공고작성 페이지
    • PostWritePage 내부에서 이전 다음 버튼을 통해 내부 컴포넌트 내용을 달리 보여주도록 구현
    • 주문매장 드롭다운 선택 기능
    • 주문할 음료를 입력할 Input 컴포넌트 추가 및 제거 기능
  • 로그인 & 공고상세페이지 & 공고작성입장페이지
    • OtherNav 컴포넌트 레이아웃 상단에 적용
  • 공고현황 페이지
    • 추후 필터링 기능을 넣기 위해 좌우 스크롤이 가능한 매장 버튼 컴포넌트 FilterForm 컴포넌트 구현
    • 상위 컴포넌트로부터 data를 전달받아 내용에 따라 공고를 달리 보여주도록 Cards 컴포넌트 구현
    • 레이아웃 하단에 공고작성페이지로 연결되는 버튼 추가

그 외 +@...

5주차 개발 내용과 관련된 PR 내역

#23 #25 #26 #28
#31 #32 #33 #35 #36 #37

5주차 개발 사항 마무리 & 리팩토링 중 변경 사항

  • App.js 에서 사용되는 페이지 및 일부 컴포넌트 내 페이지 라우팅 경로 수정
    • 공고글과 관련된 페이지는 Post... 네이밍으로 통일
  • 로그인을 위해 임시로 MyPage를 두어 로그아웃 상태에 MyPage 접속시 로그인 페이지로 유도
  • KakaoOuathPage에서 로그인 진행중에 로더화면을 확인할 수 있게 2초의 Delay 지정(차후 삭제 예정)
  • 공고페이지 & 마이페이지에서 사용되는 전용 NavOtherNav 컴포넌트로 통합
  • PostWriteIntroPage & PostDetailPage 상단에 OtherNav 컴포넌트 적용

baegyeong and others added 30 commits September 30, 2023 18:44
[4주차] : 개발 사항 Merge(develop → master)
- 로그인 페이지 상단에 사용하는 Nav 구현
- 화살표 & 홈 아이콘 클릭시 홈 페이지로 이동
- 중앙 매칭완료 문구 추가 필요
- 매칭 여부에 따라 배경색, 마감기한 색상, 화살표 색상, 마우스 커서 모양 동적으로 변경
- 버튼 컴포넌트 추가
- Link로 navigate 지정
- src/arpis/index.js : axios 인스턴스 활용 코드 및 request & response interceptor 추가
- request 시에는 토큰을 헤더에 담아 백엔드로 요청
- response 시에 에러가 존재할 경우 특정 번대 에러에 대해 에러 처리하도록 함
- ErrorPage.jsx : 404 등의 에러에 대해 해당 페이지를 띄우는데 사용
- OrderDeadLine
- OrderInfo
- OrderRequest
baegyeong and others added 20 commits October 8, 2023 00:58
[feat] : LoginNav 컴포넌트 구현
[feat] : Card 컴포넌트 기능 보완
[fix]: 버튼 컴포넌트로 변경
[feat] : 로그인 로직 추가 & 에러 페이지 추가
[feat]: 공고 작성 페이지 navigate 기능 구현
[feat] : 공고 현황 페이지 레이아웃 & 공고글 렌더링 구현
[feat] : 공고 상세 페이지 디자인 구현
[feat]: 공고작성 페이지 일부 구현
- App.js 에서 사용되는 페이지 및 일부 컴포넌트 내 페이지 라우팅 경로 수정
ㄴ공고글과 관련된 페이지는 Post... 네이밍으로 통일
- 로그인을 위해 임시로 MyPage를 두어 로그아웃 상태에 MyPage 접속시 로그인 페이지로 유도
- KakaoOuathPage에서 로그인 진행중에 로더화면을 확인할 수 있게 2초의 Delay 지정(차후 삭제 예정)
- 공고페이지 & 마이페이지에서 사용되는 전용 Nav는 OtherNav 컴포넌트로 통합
- PostWriteIntroPage & PostDetailPage 상단에 OtherNav 컴포넌트 적용
[5주차] : 5주차 개발 코드 Merge 및 일부 리팩토링
[5주차] : 개발 사항 Merge(weekly-5 → develop)
[5주차] : 개발 사항 Merge(develop → master)
@rktdnjs rktdnjs added the 🙋‍♂️ Request Review Review Request label Oct 8, 2023
@rktdnjs rktdnjs self-assigned this Oct 8, 2023
import Swal from 'sweetalert2';
import { useNavigate } from 'react-router-dom';

const navigate = useNavigate();
Copy link

Choose a reason for hiding this comment

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

저번 PR에도 남긴거 같은데 동작하나요? 제가 알기론 동작하지 않는걸로 알아서요.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이 부분은 제가 저번에 코드를 좀 잘 못 작성했던 것 같아서 이번에 다시 한 번 추가해보았는데, 추후 모킹이나 백엔드 API 개발이 완료되서 통신을 해보고 작동이 안되면 기존에 사용했었던 window.location.href 로 변경할 것 같습니다!...

import Input from '../atoms/Input';
import PlusMinusBtn from '../molecules/PlusMinusBtn';

const shortid = require('shortid');
Copy link

Choose a reason for hiding this comment

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

esm과 comonjs 방식을 혼용하지 마세요.

const shortid = require('shortid');

const OrderInfo = () => {
const [drinkInput, setDrinkInput] = useState([]);
Copy link

Choose a reason for hiding this comment

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

react-hook-form을 사용해서 폼 컨트롤 하는건 어떨까요?

const navigate = useNavigate();

const goWritePost = () => {
navigate('/post-write-intro');
Copy link

Choose a reason for hiding this comment

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

pathname 같은건 상수화 시키는게 좋습니다.

Copy link

@yoeubi yoeubi left a comment

Choose a reason for hiding this comment

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

이번주도 고생하셨습니다.

코드 스타일은 딱히 이슈가 되는 부분은 없었습니다.

@yoeubi yoeubi merged commit 6f11bf2 into review2 Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants