Skip to content

Conversation

@sss4920
Copy link
Collaborator

@sss4920 sss4920 commented Mar 17, 2024

🚩 관련 이슈

📋 구현 기능 명세

  • test 코드 작성, 불필요 exception 제거
  • test 완료 후 메인코드에 올리고 로컬에서 테스트 진행

📌 PR Point

  • 무슨 이유로 어떻게 코드를 변경했는지
    open graph 데이터를 가져올 때 브런치의 경우 image url이 가져와지는데 fname 태그 안에 들어있는 상태로 들어와서 빈 이미지가 저장되는 버그가 있어서 수정해야했습니다.

  • 어떤 부분에 리뷰어가 집중해야 하는지
    로직 변경이 원래 로직들을 건드리지는 않았을지 확인이 필요할 것 같고, 다른 링크 중에 저장안되는 케이스가 있다면 추가해야할 것 같습니다.

  • 개발하면서 어떤 점이 궁금했는지
    일단 임시로 정규식으로 때려넣긴했는데.. jsoup식으로 조금 리팩토링 할 수 있을 것 같지만 다음 브랜치 파서 해결해보도록 하겠습니다. 코드가 이쁘지 않은 것 같기도 하고 변수명 다들 너무 헷갈리게 지은 것 같아 고민을 좀 더 해보고 수정하겠습니다.

📸 결과물 스크린샷

  • Before
스크린샷 2024-03-18 오전 2 03 14
  • After
스크린샷 2024-03-18 오전 2 04 30

🛠️ 테스트

  • 테스트

🚀 API Endpoint

  • toast/save

@sss4920 sss4920 added the 🐞 fix Something isn't working label Mar 17, 2024
@sss4920 sss4920 requested a review from mmihye March 17, 2024 17:06
@sss4920 sss4920 self-assigned this Mar 17, 2024
@jeeminyi
Copy link
Member

jeeminyi commented Apr 4, 2024

INVITE ME

@sss4920
Copy link
Collaborator Author

sss4920 commented Apr 6, 2024

INVITE ME
ㅋㅋㅋㅋ왜 여깄어

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 fix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants