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

Fe/#223 WritingPage 및 ImageUploader, TechStackSelection, QuillEditor 컴포넌트 마이그레이션 #262

Merged
merged 10 commits into from
Oct 5, 2023

Conversation

Mayreeel
Copy link
Collaborator

@Mayreeel Mayreeel commented Oct 5, 2023

🛠️ 변경사항

yarn add @aws-sdk/client-s3 short-uuid @toast-ui/react-editor @toast-ui/editor-plugin-color-syntax @toast-ui/editor-plugin-code-syntax-highlight prismjs @types/prismjs
Writing 페이지 마이그레이션
ImageUploader, TechStackSelection, QuillEditor 컴포넌트 마이그레이션
S3 업로드 방식 클라이언트 사이드에서 서버 사이드로 변경을 위한 S3 업로드 API 작성

  • short-uuid로 key 설정
  • @aws-sdk/client-s3로 이미지 업로드

QuillEditor -> Toast-UI Editor로 변경

☝️ 유의사항


👀 참고자료


❗체크리스트

  • 하나의 메소드는 최소의 기능만 하도록 설정했나요?
  • 수정 가능하도록 유연하게 작성했나요?
  • 필요 없는 import문이나 setter 등을 삭제했나요?
  • 기존의 코드에 영향이 없는 것을 확인하였나요?

yarn add react-aws-s3-typescript
s3 환경변수 추가
s3config.ts 파일 추가
imageUploader 컴포넌트 마이그레이션
yarn add react-quill highlight.js
quillEditor 마이그레이션
글쓰기 페이지 마이그레이션
Quill에디터 dynamic import
yarn add @aws-sdk/client-s3 short-uuid
S3 업로드 용 API 구현
ENV 변경
기존 클라이언트 사이드 에서 업로드 => 서버사이드 업로드 및 API 호출로 이미지 주소 콜백
이미지 최적화 용 next.config image remotePatterns 설정
크기를 모르는 이미지를 위해 fill속성 추가
Largest Contentful Paint (LCP) 요소를 위한 priority 속성 추가
yarn add @toast-ui/react-editor @toast-ui/editor-plugin-color-syntax @toast-ui/editor-plugin-code-syntax-highlight prismjs @types/prismjs
마크다운 편집 밑 HTML로 백에 전송
Quill-Editor 삭제
@Mayreeel Mayreeel self-assigned this Oct 5, 2023
Copy link
Member

@Yujin-Baek Yujin-Baek left a comment

Choose a reason for hiding this comment

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

API 파일 작성 하시느라 고생하셨습니다 🥲
코드 모두 확인 완료 했습니다!

@github-actions
Copy link

github-actions bot commented Oct 5, 2023

Unit Test Results

86 tests  ±0   86 ✔️ ±0   4s ⏱️ ±0s
17 suites ±0     0 💤 ±0 
17 files   ±0     0 ±0 

Results for commit cce2a9d. ± Comparison against base commit 1e7bc4d.

@Mayreeel Mayreeel merged commit 109e9eb into FE/#207 Oct 5, 2023
3 checks passed
@Mayreeel Mayreeel deleted the FE/#223 branch October 5, 2023 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants