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.

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

refactor: 프로토콜이 붙지 않은 url에 대해서 프로토콜을 붙여주는 로직 삭제 #229

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

smb0123
Copy link
Collaborator

@smb0123 smb0123 commented Sep 24, 2024

관련 이슈

작업 분류

  • 버그 수정
  • 신규 기능 추가
  • 프로젝트 구조 변경
  • 코드 스타일 변경
  • 기존 기능 개선
  • 문서 수정

PR을 통해 해결하려는 문제가 무엇인가요? 🚀

기존의 프로토콜이 붙지 않은 url에 대해서 프로토콜을 붙여주는 로직이 모든 예외 사항을 잡아주지 않습니다.
예시는 이슈에 있습니다 !

PR에서 핵심적으로 변경된 부분이 어떤 부분인가요? 👀

해당 로직을 삭제하였습니다 !

이런 부분을 신경써서 봐주셨으면 좋겠어요. 🙋🏻‍♂️

추후에 지원서 작성 시 url을 입력하는 부분의 input을 변경하는게 좋아보입니다.

형식

url에 대한 설명 : url
ex) 노션 : 노션 url

체크리스트 ✅

  • reviewers 설정
  • assignees 설정
  • label 설정

@smb0123 smb0123 added the refactor 리팩터링 관련 이슈 label Sep 24, 2024
@smb0123 smb0123 self-assigned this Sep 24, 2024
Copy link
Collaborator

@geongyu09 geongyu09 left a comment

Choose a reason for hiding this comment

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

아이고 이런 문제까지 생각을 못했네요.. 이슈에 나온 것 처럼 사용자가 url이 아니라 일반 글을 적어버리면 문제가 발생할 수 있긴 하겠네요
늦은 시간까지 수고 많으셨습니다!

@geongyu09
Copy link
Collaborator

추가적으로 에러 메시지에서 URL을 가져올 수 없다는 것에서, @smb0123 민보님이 제시한 문제가 원인이 맞다면, 개인적으로는 Link 태그 동작에서 문제가 있을수도 있겠다고 생각이 듭니다.
위의 경우를 단순히 Link 에서 a 태그로 변경해주어도 될 것 같다는 생각이 듭니다!
물론 이도 단순 추측일 뿐 입니다..!

Copy link
Collaborator Author

@smb0123 smb0123 left a comment

Choose a reason for hiding this comment

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

@geongyu09 저도 a 태그로 변경하면 해결되는 문제인지 궁금하네요.
지금은 빠르게 문제 해결을 해야 하는 상황이라서 일단은 이전 상태로 돌리는 게 좋을 것 같아요.

Copy link
Collaborator

@2yunseong 2yunseong left a comment

Choose a reason for hiding this comment

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

넵 좋습니다 ~
추가로 나중에 논의해보면 좋을 것 같아요. 고생하셨습니다!

@smb0123 smb0123 merged commit 1fcaf57 into main Sep 24, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 리팩터링 관련 이슈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] refactor: 프로토콜이 붙지 않은 url에 대해 프로토콜을 붙여주는 로직 삭제
3 participants