part4-1st-team / coworkers

스프린트 7기 part4 프로젝트
1 stars 4 forks source link

feat/#157 마이 히스토리 페이지 리팩토링 #158

Closed mxkxx1011 closed 1 month ago

mxkxx1011 commented 1 month ago

😎 담당자

앞에 형용사 붙여주세요 ~~ ex) 귀여운 OOO

두근두근 최민경

🧩 이슈 번호

이슈 번호 적어주세요 ㅤ

  • resolves #157

✨ 작업 내용

작업한 내용을 설명해주세요

💡 이번 pr을 올리기 전에 받은 피드백이 있나요? (있으면 체크리스트 작성)

XXX

📸 스크린샷 또는 GIF

ui 컴포넌트 개발, 페이지 퍼블리싱을 했다면 스크린샷도 올려주세요

라이트 모드

스크린샷 2024-09-28 오후 11 43 21

다크 모드

스크린샷 2024-09-28 오후 11 48 24

시연영상

https://github.com/user-attachments/assets/fd3ceac4-1383-4678-8c6f-7252e7cda8ea

✍🏻 참고사항

팀원들이 참고해야할 부분이 있다면 적어주세요

history API 는 date 별로 받아올 수 가 없고 한 번에 데이터를 다 받아와서 분류하지 않는 이상 페이지가 꽤 길어질 것 같다고 판단했습니당 그래서 일주일 간격으로 데이터를 나누고 버튼을 통해 일주일동안 완료한 히스토리를 볼 수 있도록 수정해봤습니다! 깨알 포인트지만 저번주 보는거랑 다음주 보는거랑 애니메이션이 다릅니다! 데이터가 사실 많이 바뀌진 않을 거 같아서 ssr 을 사용해봤는데 맞는 방식일지는 모르겠네요! 일단 디자인 위주로 봐주세요! 성능은 체크해보겠습니다


코드 리뷰 남길때 참고용 (pr 작성할 때 지우지 말기)

P1: 꼭 반영해주세요 (Request changes) - 리뷰어는 PR의 내용이 서비스에 중대한 오류를 발생할 수 있는 가능성을 잠재하고 있는 등 중대한 코드 수정이 반드시 필요하다고 판단되는 경우, P1 태그를 통해 리뷰 요청자에게 수정을 요청합니다. 리뷰 요청자는 p1 태그에 대해 리뷰어의 요청을 반영하거나, 반영할 수 없는 합리적인 의견을 통해 리뷰어를 설득할 수 있어야 합니다.

P2: 적극적으로 고려해주세요 (Request changes) - P2에 대해 수용하거나 만약 수용할 수 없는 상황이라면 적합한 의견을 들어 토론할 것을 권장합니다.

P3: 웬만하면 반영해 주세요 (Comment) - P3에 대해 수용하거나 만약 수용할 수 없는 상황이라면 반영할 수 없는 이유를 들어 설명하거나 다음에 반영할 계획을 명시적으로 표현합니다.

P4: 반영해도 좋고 넘어가도 좋습니다 (Approve) - 해당 의견을 반영하는 게 좋을지 고민해 보는 정도면 충분합니다

P5: 그냥 사소한 의견입니다 (Approve) - 아무런 의견을 달지 않고 무시해도 괜찮습니다

PQ: 질문입니다 (Question) - 질문에는 꼭 답을 해주세요

netlify[bot] commented 1 month ago

Deploy Preview for coworkers-1 ready!

Name Link
Latest commit 2f5a1e090de48567b8604439b5ffaebd98aca52a
Latest deploy log https://app.netlify.com/sites/coworkers-1/deploys/66f845731a0f7f0008c06def
Deploy Preview https://deploy-preview-158--coworkers-1.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.