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

[GGFE-216] 관리자 - 컴포넌트, 타입 분리 + tableFormat 업데이트 #965

Merged
merged 12 commits into from
Aug 30, 2023

Conversation

PHJoon
Copy link
Contributor

@PHJoon PHJoon commented Aug 28, 2023

📌 개요

  • 관리자 - 컴포넌트, 타입 분리 + tableFormat 업데이트

💻 작업사항


AdminTableHead

  • 테이블 헤드 컴포넌트를 분리시켰습니다.

AdminEmptyItem

  • 테이블 내용이 없는 경우 사용하는 컴포넌트를 분리시켰습니다

AdminContent

  • 테이블 내용 중 글자 수 제한으로 더보기가 필요한 경우에 사용할 수 있는 컴포넌트를 분리했습니다.

dateToStringShort

  • 기존에 데이터를 받아올 때 데이터의 요소 중 date를 state에 세팅할 때 getFormattedDateToString 함수를 사용하던 것을
  • dateToStringShort로 바꿨습니다.
  • YY-MM-DD HH:MM 형식을 반환합니다.

interface 분리

  • types 폴더 하위 파일로 다 옮겼습니다.

tableFormat의 column명 업데이트

  • 기존 관리자 페이지 중 일부 페이지에서 tableFormat의 columnName이 받아온 데이터와 달라서 테이블에 표시되지 않는 문제가 있었습니다.
  • 공지사항 관리 - 시간이 표시되지 않았음 / 알림 - 알림ID가 표시되지 않았음 등

✅ 변경로직

@PHJoon PHJoon self-assigned this Aug 28, 2023
Copy link
Member

Choose a reason for hiding this comment

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

하나 만들어두니까 좋네요👍 코인 내역 모달도 이걸로 수정해야겠네요

Copy link
Member

@hyobb109 hyobb109 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
Member

@yoouyeon yoouyeon 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
Member

Choose a reason for hiding this comment

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

패널티 관리 페이지에서 처음으로 전체기록 버튼을 누르면 아래 같은 에러가 뜨네여,,,
key는 다 잘 들어가 있는 것 같은데 왜 에러가 뜨는지는 모르겠습니다 🥲

Screen_Shot 2023-08-30 13 13 55

Copy link
Contributor Author

@PHJoon PHJoon Aug 30, 2023

Choose a reason for hiding this comment

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

빈 태그를 <div key={index}></div>로 변경하여 해결하였습니다!

Copy link
Member

Choose a reason for hiding this comment

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

앗 빈 태그가 문제였던거면 div 대신 <React.Fragment> 를 쓰는것도 좋을 것 같아요! (https://react.dev/reference/react/Fragment)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fragment 배워갑니다! 변경했습니다!

@PHJoon PHJoon merged commit 0c84604 into main Aug 30, 2023
1 check passed
@PHJoon PHJoon deleted the GGFE-216-관리자-컴포넌트-css-리팩토링 branch August 30, 2023 06:24
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