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

PEOPLE 페이지 기수 컴포넌트 개선 #66

Merged
merged 1 commit into from
Oct 13, 2024

Conversation

seoooa
Copy link
Collaborator

@seoooa seoooa commented Oct 13, 2024

✅ Checklist

  • 올바른 브랜치에 PR을 보내도록 설정하였나요?
  • PR의 라벨을 올바르게 달았나요?

✨ 현재 작업은 어떤 이슈를 해결한 것인가요?

관련된 이슈 number: #65

  • 기수 정렬 오류 수정했습니다.
  • 운영진 이모지 제거하고, 간격 조정했습니다.
  • 운영진이 있을 때에만 노출되게 수정했습니다.

기존 코드에서 변경된 사항

  • 있음
  • 없음
  1. 이모지에 사용했던 mockImage 제거했습니다.
  2. 기수 탭을 오름차순 정렬하였습니다.

@seoooa seoooa self-assigned this Oct 13, 2024
@seoooa seoooa added bug 버그 feature 기능 구현 labels Oct 13, 2024
@seoooa seoooa changed the title 기수 컴포넌트 개선 PEOPLE 페이지 기수 컴포넌트 개선 Oct 13, 2024
@rmdnps10
Copy link
Contributor

LGTM ! 고생하셨습니다:)

@rmdnps10 rmdnps10 merged commit 7e2b24c into main Oct 13, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 버그 feature 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants