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

Feature/메인페이지 동아리연혁 #123 #474

Merged
merged 15 commits into from
Aug 12, 2023

Conversation

jasper200207
Copy link
Collaborator

@jasper200207 jasper200207 commented Jul 22, 2023

연관 이슈

작업 요약

메인페이지의 동아리연혁 파트 입니다

작업 상세 설명

  • HistoryItem으로 컴포넌트 분리하여 구성하였습니다
  • api로 받아온 데이터중 항목이 없는 년도를 제외하고 역순으로 배치하였습니다

리뷰 요구사항

  • 10분
  • Feature/메인페이지 동아리 자랑 #122 #471 선행리뷰 부탁드립니다.
  • 다른 브렌치 내용이 섞여 들어갔는데 History.jsx 파일만 확인해주시면 됩니다
  • 원과 직선으로 구성된 디자인을 div와 border로 구현하였는데 디자인 의도에 맞는지 확인 부탁드립니다

Preview 이미지

스크린샷 2023-07-22 오후 11 56 05

jasper200207 and others added 15 commits April 2, 2023 20:03
…nt-R2 into feature/메인페이지_동아리_자랑_#122
- api 연결했습니다
- 이미지는 프론트에서 저장하기로 해서 이미지 추가했습니다
- activity의 이미지도 프론트로 위치 바꿨습니다. 이미지가 있는 경우 960px로 높이를 고정시키니 칸이 안맞는 문제가 생겨서 높이고정은 이미지가 없는 경우에만 적용되도록 수정했습니다
- #122
…nt-R2 into feature/메인페이지_동아리_자랑_#122
- 불러온 api 내용 ui 반영
- #122
…nt-R2 into feature/메인페이지_동아리연혁_#123
- HistoryItme으로 컴포넌트를 만들어 분리
- #123
@jasper200207 jasper200207 self-assigned this Jul 22, 2023
@jasper200207 jasper200207 requested review from hyuunnn, lcqff and Ji-Hyeon212 and removed request for hyuunnn July 22, 2023 14:57
@jasper200207 jasper200207 added the Feature 기능 추가 label Jul 22, 2023
@publdaze publdaze added the P1 급하게 머지해달라 요구할 때 사용 label Aug 2, 2023
Copy link
Contributor

@lcqff lcqff left a comment

Choose a reason for hiding this comment

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

굿입니다!👍

Copy link
Contributor

@Ji-Hyeon212 Ji-Hyeon212 left a comment

Choose a reason for hiding this comment

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

버티컬 타임라인 ui 는 어떻게 구현하는 지 궁금했는 데 ul 태그 사용해서 구현할 수 있군요! 알아갑니다!

@jasper200207 jasper200207 merged commit 9a323d2 into develop Aug 12, 2023
@jasper200207 jasper200207 deleted the feature/메인페이지_동아리연혁_#123 branch August 12, 2023 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature 기능 추가 P1 급하게 머지해달라 요구할 때 사용
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants