회원가입 약관 동의, 약관 상세

[kd02109]

What is this PR?

• 회원 가입 약관 동의 페이지 구현
• 회원 가입 약관 동의 상세 설명 페이지 구현

---

What has changed?

• 우선 기본 화면 구성에 집중했습니다.

약관동의

• /agreement url 주소에 약관 동의 페이지 화면을 만들었습니다.
• 아직 button 컴포넌트 디자인 수정이 안되어 있어서 기존 컴포넌트를 활용했습니다.
• BottomSheet가 구현이 안되어 있기 때문에, 차후 활용하는 것을 가정해서 client component로 분리하여 두었습니다.

약관 동의 상세

• /agreement/detail url 주소에 페이지를 구현했습니다.
• X 버튼을 클릭하면 /agreement url로 가도록 했습니다.
• 해당 페이지 이동을 고려한다면, 약관동의 여부를 context 혹은 상태 공유 api를 통해 관리 해야 할 것 같습니다.

---

A notice to reviewers...

---

Screen

• agreement

• detail

[ho991217]

고생하셨습니다 🔥
이것 저것 달다보니 리뷰가 18개가 되어버렸네요...!
활발한 의견 공유 있었으면 좋겠습니다!

[ho991217]

Uncaught ModuleBuildError: Module build failed (from ../../node_modules/@vanilla-extract/webpack-plugin/loader/dist/vanilla-extract-webpack-plugin-loader.cjs.js):
NonErrorEmittedError: (Emitted value instead of an instance of Error) ReferenceError: $RefreshReg$ is not defined

라는 에러가 뜨면서 접근이 안되네요

[kd02109]

어 저는 해당 문제 없이 dev 환경에서 실행이 되는데, 빌드시에 발생된 에러인가요?

[ho991217]

아뇨 dev 모드로 실행후 /agreement/detail 접근하면 발생합니다

[DoctorK4]

저도 /agreement/detail 엔드포인트에서 접근이 안됩니다.

[ho991217]

여전히 안되네요 ;(

[kd02109]

해당 부분은 저는 정상적으로 동작을 해서 살펴보도록 하겠습니다. vanilla-extract-css/vanilla-extract#1043 해당 이슈를 참고하면 module 방식과 연관이 있는 것 같은데, 저는 에러가 구현이 안 되서 해결 여부를 확인할 수가 없네요.. ㅜㅜ

[kd02109]

해당 review 반영해서 수정해야 하는 부분 수정했습니다!

[kd02109]

url 경로를 하나의 object로 관리하면 좋을 것 같은데 /app/_lib 폴더를 만들고 하나의 파일에서 관리를 하려고 합니다. path.ts로 파일을 만들어서 path 변수에서 관리하는게 어떨까요? 추가 path가 있을 때 마다 path 변수에 url을 추가하면서 관리하면 좋을 것 같습니다.!

[DoctorK4]

url 경로를 하나의 object로 관리하면 좋을 것 같은데 /app/_lib 폴더를 만들고 하나의 파일에서 관리를 하려고 합니다. path.ts로 파일을 만들어서 path 변수에서 관리하는게 어떨까요? 추가 path가 있을 때 마다 path 변수에 url을 추가하면서 관리하면 좋을 것 같습니다.!

좋습니다!

[ho991217]

url 경로를 하나의 object로 관리하면 좋을 것 같은데 /app/_lib 폴더를 만들고 하나의 파일에서 관리를 하려고 합니다. path.ts로 파일을 만들어서 path 변수에서 관리하는게 어떨까요? 추가 path가 있을 때 마다 path 변수에 url을 추가하면서 관리하면 좋을 것 같습니다.!

이름은 _lib이 적당할까용? 상수라는 의미로 /constants/path.ts 도 좋을 것 같구요

[ho991217]

여전히 안되네요 ;(