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

13-4 [FE] [논문 상세 - 논문 정보] 검색한 논문의 제목, 저자, DOI, 인용논문 목록을 보여주고, DOI 클릭 시 논문 원본으로 이동한다. #76

Merged
merged 9 commits into from
Nov 30, 2022

Conversation

Palwol
Copy link
Collaborator

@Palwol Palwol commented Nov 28, 2022

개요

  • 논문 상세페이지 접속 시 논문 정보 api를 불러와서 상세정보를 보여줍니다.
  • DOI 클릭 시 논문 원본으로 이동합니다.

작업사항

  • 논문 상세페이지 레이아웃 구현
  • 상세페이지 url 쿼리로 논문 정보 api 호출
  • DOI 클릭 시 논문 원본 페이지로 이동
  • 뒤로가기 버튼 클릭 시 이전 페이지로 이동
  • 로고 클릭 시 메인 페이지로 이동

참고사항(예외 처리)

  • reference에 title이 없으면 보이지 않도록 설정했습니다.
  • reference들의 key값을 우선 index로 설정해 두었습니다. 추후 crossref api의 doi(없을 경우 key)로 변경 예정입니다.
  • reference가 없는 논문은 서버 에러가 발생합니다. 서버 코드 수정 예정입니다.
    • reference의 DOI가 없을 때 disabled 처리는 아직 구현하지 않았습니다.
  • api 호출 시 에러 처리는 추후 구현 예정입니다.

스크린샷

상세페이지

리뷰 요청사항

  • 현재 컴포넌트 구조는 PaperDetail이라는 상위 컴포넌트에서 데이터를 불러와서 하위 컴포넌트(좌측 논문 정보, 우측 그래프 컴포넌트)에 데이터를 내려주는 방식입니다. 로직 분리가 적절한지, 더 좋은 구조가 있을 지 조언 부탁드립니다.
  • 참고사항의 예외 처리 이외에 추가로 예외 처리가 필요한 부분이 있을 지 조언 부탁드립니다.

frontend/src/icons/LogoIcon.tsx Outdated Show resolved Hide resolved
frontend/src/pages/PaperDetail/PaperDetail.tsx Outdated Show resolved Hide resolved
frontend/src/pages/PaperDetail/PaperDetail.tsx Outdated Show resolved Hide resolved
frontend/src/constants/path.ts Outdated Show resolved Hide resolved
Copy link
Collaborator

@JunYupK JunYupK left a comment

Choose a reason for hiding this comment

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

화면 캡처 2022-11-29 113826

다음과 같은 axois 500대 에러가 자주 발생 하네요.

frontend/src/pages/PaperDetail/components/PaperInfo.tsx Outdated Show resolved Hide resolved
@Palwol
Copy link
Collaborator Author

Palwol commented Nov 29, 2022

리뷰사항 적용 완료했습니다.

추가사항

  • 공통 컴포넌트들이 많아지면서 components 폴더가 혼란스러운 것 같아 Search 컴포넌트와 그 자식 컴포넌트들을 components 폴더 내의 search 폴더로 이동시켰습니다.
  • 자동완성 검색어 클릭 시 논문 상세 페이지로 이동하는 기능을 추가했습니다.

@yeynii yeynii self-requested a review November 29, 2022 10:08
Copy link
Member

@yeynii yeynii left a comment

Choose a reason for hiding this comment

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

확인 완료했습니다.

@yeynii yeynii closed this Nov 29, 2022
@yeynii yeynii reopened this Nov 29, 2022
Copy link
Collaborator

@JunYupK JunYupK left a comment

Choose a reason for hiding this comment

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

확인 완료했습니다. 고생하셨습니다.

@Palwol Palwol merged commit bea4df2 into dev Nov 30, 2022
@Palwol Palwol deleted the feature/paper-detail branch December 1, 2022 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants