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.

Sign up for GitHub

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

Jump to bottom

Webpack 배포 설정, AWS 배포 체크 #139

Merged
merged 8 commits into from
Jul 25, 2024
Merged

Webpack 배포 설정, AWS 배포 체크 #139

merged 8 commits into from
Jul 25, 2024

Conversation

Jaymyong66
Copy link
Contributor

@Jaymyong66 Jaymyong66 commented Jul 25, 2024 •
edited
Loading

⚡️ 관련 이슈

close #129, #138

📍주요 변경 사항

  1. webpack config 파일을 mode에 따라 common, dev, prod로 분리
  2. 환경 변수 파일 .env를 .env.development, .env.production 으로 분리
  3. react-router의 basename 제거
  4. file-loader 패키지 제거
  5. 백엔드 서버와 연결
  6. 환경 변수가 API_NAME인 변수명을 BASE_NAME -> API_NAME으로 일관되게 변경

🎸기타

  1. 현재 배포 페이지에서 favicon 경로 에러가 납니다. 추후 fix 하도록 해야겠습니다.
  2. www.code-zap.com 도메인으로 접근 가능합니다.
  3. 관련 트러블 슈팅과 개발 문서는 정리해두었습니다.

@Jaymyong66
chore: webpack의 file-loader 제거, webpack5의 설정으로 대체
26365ed
@Jaymyong66
chore: webpack의 설정을 common, dev, prod로 파일 변경
538def1 
해당 과정에서 dotenv 패키지를 dotenv-webpack 패키지로 변경
브라우저 환경에서 process undefined 에러를 해결하기 위해 process 패키지 설치
@Jaymyong66
refactor(api): BASE_URL -> API_URL 변수명 변경
a29e046 
환경변수의 변수명에 맞춤
@Jaymyong66
refactor(routes): 환경변수 파일 분리에 따른 BASE_URL 분기처리 제거
13a4678
@Jaymyong66
refactor(routes): basename 제거
ad64311
@Jaymyong66
chore: webpack.common.js로 파일명 변경, clear 옵션 추가
365606d
@Jaymyong66
chore: publicPath 옵션을 webpack.dev.js로 변경
5641f5d
@Jaymyong66
chore: process 패키지 제거
ec57fbb 
process 패키지는 배포(브라우저) 환경에서 환경변수를 위한 process 모듈을 찾지 못하는 에러를 잡기 위함이었다.
다만 dotenv-webpack의 공식 문서의 방식인 ignoreStuyb : ture 옵션으로 충분히 해결 가능하다고 판단.
@Jaymyong66 Jaymyong66 added feature 기능 추가 FE 프론트엔드 labels Jul 25, 2024
@Jaymyong66 Jaymyong66 added this to the 2차 스프린트 🐣 milestone Jul 25, 2024
@Jaymyong66 Jaymyong66 self-assigned this Jul 25, 2024
Hain-tain
Hain-tain approved these changes Jul 25, 2024
View reviewed changes
Copy link
Contributor

@Hain-tain Hain-tain left a comment

Choose a reason for hiding this comment

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

굿! 고생하셨습니다

@vi-wolhwa vi-wolhwa merged commit 7e0bb4b into woowacourse-teams:dev/fe Jul 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hain-tain Hain-tain Hain-tain approved these changes

@vi-wolhwa vi-wolhwa vi-wolhwa approved these changes

Assignees

@Jaymyong66 Jaymyong66

Labels
FE 프론트엔드 feature 기능 추가
Projects
2024-code-zap
Status: Weekend Done
Milestone
2차 스프린트 🐣
Development

Successfully merging this pull request may close these issues.
3 participants
@Jaymyong66 @vi-wolhwa @Hain-tain