-
Notifications
You must be signed in to change notification settings - Fork 1
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
[Feature] client, admin layout 구축 #15
Conversation
@@ -4,7 +4,6 @@ | |||
"outDir": "dist", | |||
"baseUrl": ".", | |||
"paths": { | |||
"@/*": ["src/*"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저도 옛날에 회사에서 일했을때 절대경로가 안먹혀서 전체 다 상대경로로 바꾸었던 기억이...😭
만약 packages/ui
<-- 요거를 빌드한 결과물을 사용하게끔 하려면 절대경로로 사용할 수 있는데, 빌드한 결과물이 아닌 것을 활용하게 하려면 상대경로 쓰는게 불가피한것 같기도합니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵 그런 거 같네요...
이건 다음 회의 때 한 번 얘기해봐도 좋을 거 같아요!
전 둘다 상관 없을 거 같아요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저는 개인적으로 빌드하지 않고 사용하는걸 선호하긴 합니다..ㅎㅎ
ui바꾸고 잘 되는지 테스트해보려면 빌드 기다리는게 좀 오래걸리더라구용.
요거 회의때 이야기해보아요~!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고 많으셨어요~!!
package/ui 쪽에 admin/client 분리해주셨는데 추후에 비즈니스 로직과 얽혀있는 컴포넌트를 만들 때에도 package/ui 쪽에 관리되는 게 좋을까요..?! context 나 여러 가지 상태 로직들은 apps 각 서비스 폴더에서 관리하는 것도 좋을 거 같아서요..!
위 같은 경우가 생긴다면 package/ui 쪽에는 비즈니스 로직과 관련없는 뷰 관련 컴포넌트만 관리되는 게 좋을 것 같아요. 다른 분들은 어떻게 생각하시나요? 🤔
음 저도 그게 나을 거 같네요! |
저도 세은님 의견에 동의합니다! ✨ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
머지머지~~
🎉 변경 사항
client, admin layout 구축했습니다.
🚩 관련 이슈
🙏 여기는 꼭 봐주세요!
ui 패키지 내부에도 apps/admin, apps/client와 동일하게 절대 경로 세팅이 되어 있었는데, 서로 resolve해야 하는 경로가 다르다 보니 부딪혀서 ui 패키지에서는 상대 경로만 사용해야 할 것 같습니다...
일단은 상대 경로로 사용하고 있어요
ui 쪽에 admin, client, shared 폴더 만들어서 Navbar 관련 컴포넌트 넣어뒀습니다.
그리고 apps/admin, apps/client next.config 들어가보면 홈으로 접속 시 /studies, /my-study로 리다이렉트되도록 설정 추가해놨습니다.
폰트 세팅도 같이 해놨습니다.
그리고 ui 쪽에서 스타일 변경 사항 있으면 generate:css-file 스크립트 동작시켜서 반영해야 apps에서도 동일하게 볼 수 있습니다.