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

[feat, chore] 자잘한 것들 수정 및 성능 개선 #115

Merged
merged 13 commits into from
Aug 19, 2024

Conversation

lybell-art
Copy link
Collaborator

@lybell-art lybell-art commented Aug 19, 2024

#️⃣ 연관 이슈

📝 작업 내용

햄버거 버튼을 추가했습니다.

  • 애니메이션은 구현 안 했습니다.

이미지 lazy loading을 적용했습니다.

스크린샷 2024-08-19 오전 11 53 46

(이게 적용 전)

스크린샷 2024-08-19 오후 12 22 18

(이게 적용 후)

적용 전에 비해, LCP가 5초 이상 개선되었습니다.
로컬에서 FCP 5초 지연 문제는 모르겠네요. SSG인데 대체 왜!!!

서버와의 응답 상태와의 호환을 맞췄습니다.

  • 이제 기대평 Info api가 401 에러를 띄우더라도, 기대평 버튼이 "error"로 표시되지 않습니다.
  • 로그인한 사용자가 추첨 이벤트를 참여하지 않았을 때, 기대평 등록 api에서 404 에러를 띄우는데, 이 경우에도 추첨 이벤트로 유도하는 모달의 띄워지도록 했습니다.

이제 사용자 이름은 제대로 표시됩니다.

  • jwt 파서를 적용했습니다.

메인 페이지의 로그인 상태 불러오기를 개선했습니다.

  • 진짜 아주 찰나지만, 이제 메인 페이지의 로그인 상태는 '즉시' 반영됩니다.
  • 즉, 접속 시 로그인 상태가 false에서 true로 바뀌는 현상이 없기 때문에, 로그인 상태에 의존하는 api가 더 이상 2번 호출되지 않습니다.

프리뷰 서버는 이제 진짜 백엔드 서버로 api를 발송합니다.

  • 이제 dev 브랜치로 머지하지 않아도 백엔드 서버와의 연동을 확인할 수 있습니다. 물론 빌드를 일일이 해야 하는 건 똑같긴 하지만요...

자잘한 오류를 수정했습니다.

  • 리셋 버튼에 aria-label을 추가했습니다.

다음에 할 것

  • 추첨 이벤트 참여 상태는 이제 전역으로 관리될 예정입니다.
  • 인터랙션 페이지의 접근성이 개선될 예정입니다.
  • 모달의 접근성이 개선될 예정입니다. (�focus trap 적용 예정)
  • 어드민 페이지에서 이벤트 프레임 입력에 검색 미리보기 기능이 적용될 예정입니다.

@lybell-art lybell-art added fix 버그 및 오류 수정 feat 기능 구현 refactor 코드 리팩토링 labels Aug 19, 2024
@lybell-art lybell-art requested a review from darkdulgi August 19, 2024 10:23
@lybell-art lybell-art self-assigned this Aug 19, 2024
Copy link
Collaborator

@darkdulgi darkdulgi left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@darkdulgi darkdulgi merged commit c85ff9c into dev Aug 19, 2024
1 check passed
@lybell-art lybell-art deleted the feature/performance-upper branch August 28, 2024 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 구현 fix 버그 및 오류 수정 refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants