Skip to content

Conversation

@gudusol
Copy link
Contributor

@gudusol gudusol commented Apr 12, 2025

📌 Related Issue Number


Checklist

  • 🎋 base 브랜치를 제대로 설정했나요?
  • 🖌️ PR 제목은 형식에 맞게 잘 작성했나요?
  • 🏗️ 빌드는 성공했나요? (pnpm build)
  • 🧹 불필요한 코드는 제거했나요? e.g. console.log
  • 🙇‍♂️ 리뷰어를 지정했나요?
  • 🏷️ 라벨은 등록했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 안쓰는 페이지 삭제
  2. 토큰 관리 방식 변경 (이상하게 잘못된 컨텍스트 방식 -> 클로저 함수로 저장)
  3. 로그인 여부 확인
  4. 각종 페이지에 로고 수정
  5. 코드 리팩토링
    • path alias 수정
    • components, utils 도메인별 분리
    • svg 원본파일 삭제
    • 함수 params 잘못된 부분 수정
    • 안쓰는 함수 정리

💡 New Insights & Learnings

  • 추가로 번들파일 크기 확인하는 것도 추가해둠.
  • dis/stats.html에서 확인할 수 있고, 현재 lodash 등이 필요없이 번들 파일을 차지하고 있는 부분이 있는 것 같아서 추후 수정할 예정

@gudusol gudusol added 🔨 Refactor 코드 리팩토링 🧹 Chore 패키지 매니저 수정, 그 외 기타 수정 🐽 태승 labels Apr 12, 2025
@gudusol gudusol self-assigned this Apr 12, 2025
@gudusol gudusol merged commit 46fadf2 into main Apr 12, 2025
1 check passed
@gudusol gudusol deleted the feat/admin/refactoring-#68 branch April 12, 2025 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐽 태승 🧹 Chore 패키지 매니저 수정, 그 외 기타 수정 🔨 Refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feat] 관리자 리팩토링

2 participants