ryush00 / capstone

1 stars 0 forks source link

회원가입 두껍게, 확인 이메일 재전송 창 문구변경 #47

Closed Yang0Jin0Woo closed 1 week ago

Yang0Jin0Woo commented 1 week ago

작업 내용

스크린샷

리뷰 시 참고사항

기존 페이지 입니다.

새벽에 PR 죄송합니다..

ryush00 commented 1 week ago

46 과 관련하여

해당 문제는 Turbo라는 자바스크립트쪽 프레임워크 문제라서 조금 더 깊게 들어가봐야 합니다.

지금 수정해주신 내용을 보면 새로고침을 하게 코드를 변경해 주신 것으로 보이는데, 이는 Turbo 프레임워크와 동작이 꼬일 우려가 있으며.. 약간 구멍난 항아리를 반창고로 막은 느낌입니다.

https://github.com/ryush00/capstone/pull/47/commits/20dc1306c567e7fc856e6bb04a901f935a0c64cc#diff-cfd4def2fa1b6d7a815550e929b44aa63c43e5deddd28df8ecd2f9d8882a8f7bR293

아니면 아예 이 프레임워크를 제거할 수도 있고요.

이 이슈는 제가 나중에 이 프레임워크를 제거하거나 해결책을 찾아보고자 하니, 이 PR에서는 수정하지 말아주실 수 있을까요?

Yang0Jin0Woo commented 1 week ago