-
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
[chore] next.js CI Github Action 완성 #177
Conversation
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.
수고하셨습니다! 이거 머지해도 되는 건가요 ??
결론부터 말씀드리면, 아직 안정적인 상태는 아닙니다. 제가 해당 PR을 DRAFT로 변경하려고 ORGANIZATION 권한을 요청드렸는데, 가능하시다면 권한을 부여해 주시면 감사하겠습니다. 현재 사이트 접속 시 알 수 없는 오류가 발생하여 조사 중입니다만, 제가 Next.js에 대한 경험이 부족하여 아직 원인을 파악하지 못하고 있습니다. 정상적으로 로그인 및 백엔드 통신을 확인한 후에 병합을 진행하고자 합니다.
|
본 레포지토리에서는 이미 admin으로 되어 있으셔서, 어느 부분에서 권한을 드려야하는지 잘 모르겠네요.. ! 그냥 PR 말머리를 [draft] 로 바꿔주셔도 괜찮을 것 같습니다! 말씀하신 이슈에 관해서는 저도 동일한 이슈가 뜹니다. 저도 처음 보는 에러라 좀 더 찾아봐야 할 것 같습니다. 로그인과 관련해서는 const result = await signIn('credentials', {
accountId: form.accountId,
password: form.password,
redirect: false,
}); 부분에서 실행하고 있습니다. NextAuth의 signIn 함수를 사용하고 있어서, 저도 해당 이슈 방법을 찾아보겠으나, 계속 이슈가 유지된다면 멘토링 요청도 필요할 것 같습니다! 고려해보시고 진행 부탁드립니다. 이슈 관련해서 알아낸 게 있으면 연락드리겠습니다! |
관련 에러 모두 해결하였고 정상적으로 로그인 되는 것 확인하였습니다! 또한 현재 dev 브랜치를 main 으로 머지해도 괜찮을까요? |
노션 확인했습니다! 고생 많으셨군요... 🥲 해결돼서 다행이에요! 머지 진행하면 될 것 같아서, 본 PR 머지하도록 하겠습니다! dev 브랜치 머지는 오늘 밤(~새벽) 작업 후에 진행하고 싶은데 괜찮을까요?? |
넵 편하실때 해주시면 됩니다 항상 수고 많으십니다 ㅎㅎ |
📑 구현 사항
main 브랜치에 pull_request 를 통해 머지될 경우 깃액션이 돕니다.
yml과 Dockerfile을 npm sharp 을 통해 최적화하였습니다.
각 배포 시 마다, Release 버전이 자동으로 업데이트 됩니다.
KCR까지 이미지가 정상적으로 업로드되면 성공입니다.
🌁 결과 화면
과정 살펴보기
연관 이슈
코멘트