[Feat] 해시태그 api 연결

[sispo3314]

🔍 관련된 이슈

• closed [Feat] #47 해시태그 API 연결 #47

📝 작업 내용

• 해시태그 타입을 스웨거와 통일
• 해시태그 API 연결
• 충돌 수정

📸 스크린샷

🚨 이슈

📣 리뷰 요구사항

파일 분리나 가독성면에서 분리하면 좋을 부분이 있다면 말씀해주세요! 더 좋은 방법이 있다면 코멘트 부탁드립니다!

✅ 체크리스트

• 코드가 정상적으로 컴파일되나요?
• merge할 브랜치의 위치를 확인했나요?
• Label을 지정했나요?
• 리뷰어를 지정했나요?

[dalzzy]

수고하셨습니다~ 👍🏻 api 연결 처음이신 것 같은데 엄청 빨리 구현해주셧네용 다 잘 구현해주신 것 같습니다

git origin pull develop을 한 후에 ENOENT: no such file or directory, open 'src/assets/icons/header_logo.svg'
라는 오류가 발생했는데 해당 코드가 어디에 있는지 찾을 수가 없어서... PR 올린 후 해결하겠습니다!
=>아마 이전 PR 머지에서 발생한 문제 같아서 우선 pull 취소하고 다시 push했는데 빌드 오류 해결하려면 pull 해와야 하는 상황입니다 ㅜ.ㅜ

이 부분은 아마 캐싱 문제일 수 있는데 서버를 껐다가 다시 npm run dev 해보시겠어요?
제가 리팩토링하면서 중복 아이콘 없앴는데 저도 실행 시 똑같은 오류가 발생했어서.. 서버 껐다 다시 키니까 되더라구요,,!

추가로 API 연동 관련 PR을 올릴 때는,
요청이 성공했는지 확인할 수 있는 콘솔이나 개발자 도구 > 네트워크 탭의 응답 스크린샷을 함께 첨부해주시면 좋을 것 같습니다!

예시사진을 첨부해드리자면,
요청한 API에 대해 아래와 같이 응답이 오는 걸 확인할 수 있습니다 🙌

[sispo3314]

@dalzzy 해당 문제는 다시 pull하니까 해결 되었습니다 감사합니다!! 그런데 지금 확인해보니까 헤더 공통 컴포넌트에 배경색이 들어가면서 리뷰디테일 페이지가 디자인 요구사항과 조금 달라졌는데 제가 className을 prop에 추가하는 방식으로 수정해도 될까요??

이게 현재 보여지는 페이지고 아래가 피그마입니다!

[dalzzy]

@dalzzy 해당 문제는 다시 pull하니까 해결 되었습니다 감사합니다!! 그런데 지금 확인해보니까 헤더 공통 컴포넌트에 배경색이 들어가면서 리뷰디테일 페이지가 디자인 요구사항과 조금 달라졌는데 제가 className을 prop에 추가하는 방식으로 수정해도 될까요?? 이게 현재 보여지는 페이지고 아래가 피그마입니다!

앗 그러네용...ㅜㅜ 저도 리팩토링하면서 고민하긴했었는데,,
다른페이지에서도 굳이 헤더를 고정시키지 않아도 될 것 같기도 해서 그냥 배경색이랑 fixed로 위치 고정시킨거 없애도 괜찮을거같긴합니다.....
일단 오늘 회의에서 디자이너분들께 한번 여쭤볼게요! 만약 헤더 고정해달라고 하시면 말씀해주신 대로 디테일 페이지 헤더에서는 배경색 없애도록 수정해주시면 될 것 같아요!

[dalzzy]

스크린샷 첨부하신거까지 확인했습니당 ~~👏🏻

[soyun-git121]

api연결 고생하셨습니당 코드들 저도 작업할때 참고해보겠습니다~!!