-
Notifications
You must be signed in to change notification settings - Fork 4
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
자동 배포 구현 #419
Closed
ezcolin2
wants to merge
68
commits into
boostcampwm-2024:develop
from
boostcampwm-2024:refactor-be-#33
Closed
자동 배포 구현 #419
ezcolin2
wants to merge
68
commits into
boostcampwm-2024:develop
from
boostcampwm-2024:refactor-be-#33
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 미사용 기능이므로 제거
- horizontal -> vertical
- 동일한 의존성을 가지고, 같은 의미의 로직을 수행하므로 합침
- 논의가 필요한 TODO는 노션에 따로 표시
Co-authored-by: Summer Min <[email protected]> Co-authored-by: ez <[email protected]>
Co-authored-by: ez <[email protected]> Co-authored-by: Summer Min <[email protected]>
Co-authored-by: 박경희 <[email protected]>
- currentPage를 이용한 EditorView 분기 처리 추가
refactor: 도커 최적화 - 중복되는 볼륨 처리
FCP 개선을 위한 사용하지 않는 js 코드 삭제
Co-authored-by: baegyeong <[email protected]>
Co-authored-by: baegyeong <[email protected]>
- rounded 적용 안됨 이슈 해결결 Co-authored-by: baegyeong <[email protected]>
Co-authored-by: baegyeong <[email protected]>
lazy loading 적용
lighthouse 접근성 개선
- authQueries와 동일한 커스텀훅이 있어서 삭제 Co-authored-by: 박경희 <[email protected]>
Co-authored-by: 박경희 <[email protected]>
- 옥토독스 서비스 한줄소개 추가 Co-authored-by: 박경희 <[email protected]>
Co-authored-by: 박경희 <[email protected]>
Best Practice 개선
- node_modules를 가지고 있는 docker volume 모두 제거 - root와 각 workspace별 node_modules를 가진 이미지 생성 - 위에서 만든 이미지를 기반으로 container 여러 개 생성
- redis에 page content 변경 사항을 저장하는 key와 분산 락을 저장하는 key가 중복 발생 - key를 다르게 변경
- 빌드 스테이지에서 빌드 진행 - 프로덕션 스테이지에서 node_modules 가지고 있는 이미지를 base로 사용
- backend, websocket 빌드 스테이지 기반 이미지 octodocs-modules로 변경 - 로컬 nginx 환경 conf.d 디렉토리 마운트에서 conf.d/default.conf 마운트로 변경
- 백그라운드와 포그라운드 대비 개선을 위해 수정 Co-authored-by: 박경희 <[email protected]>
- 제대로 production 환경에서 적용되지 않아서 삭제 Co-authored-by: 박경희 <[email protected]>
- 실제 배포 환경에서 콘솔 에러를 삭제하고자 세팅 Co-authored-by: 박경희 <[email protected]>
node_modules를 가진 base 이미지로 서버 이미지 빌드
Co-authored-by: 박경희 <[email protected]>
Co-authored-by: 박경희 <[email protected]>
isLoading 상태 제거, 사용하지 않는 파일 삭제
- frontend lint 설정에서 임시적으로 any 허용 - prettier/prettier가 중복 설정되어 발생한 오류 수정 - lint 적용
hotfix: lint 오류 수정
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔖 연관된 이슈
📂 작업 내용
📑 참고 자료 & 스크린샷 (선택)
📢 리뷰 요구사항 (선택)