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

refactor : 일기 작성페이지 리펙토링 #449

Merged
merged 38 commits into from
Dec 30, 2024

Conversation

HelloWook
Copy link
Member

@HelloWook HelloWook commented Dec 27, 2024

🚀요약

  • 일기 작성페이지 리펙토링

📝작업 내용

  • 구조 분리
  • localstorage -> indexdb로 마이그레이션
  • 에러 바운더리 구축

🔍 전달 사항

상세 페이지도 리펙토링이 필요하다 생각되는데 이 부분은 나중에 상의가 조금 필요할듯 싶습니다.

🎸기타 (연관 이슈)

close #8

Copy link

github-actions bot commented Dec 27, 2024

✅ Build 성공!

Copy link

github-actions bot commented Dec 27, 2024

⚡️ Lighthouse Report (Average of 5 runs)

Category Score
🟠 Performance 84
🟠 Accessibility 68
🟢 SEO 90
🟢 Best Practices 100

Core Web Vitals (Average)

Metric Value
🟠 First Contentful Paint 2.7 s
🟠 Largest Contentful Paint 3.1 s
🟢 Cumulative Layout Shift 0.0

import React from 'react';
import { LetterInput } from '../LetterInput/LetterInput';

type LettetProps = {
Copy link
Collaborator

@cmlim0070 cmlim0070 Dec 28, 2024

Choose a reason for hiding this comment

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

p4: LetterProps인데 오타가 난 것 같아요...!

Copy link
Member Author

Choose a reason for hiding this comment

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

앗 바로 수정하겠습니다..!

웹 접근성 향상을 위한 aria-lable 작성
Copy link

@HelloWook HelloWook merged commit 660fc24 into develope Dec 30, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 리펙토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat : error boundary 설정
2 participants