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

푸시 알림 클릭시 이동 문제 확인 #681

Merged
merged 1 commit into from
Oct 18, 2024

Conversation

pricelees
Copy link
Contributor

@pricelees pricelees commented Oct 17, 2024

PR의 목적이 무엇인가요?

백엔드에서 푸시 알림 클릭시 이동 경로를 보내주는 방법을 수정해서, 프론트 코드에 반영했어요.

이슈 ID는 무엇인가요?

설명

테스트를 하는 도중 뒤로가기 버튼을 눌렀을 때 이동하지 않아서 몇 시간을 헤맸는데..
최신 코드가 아니어서 그랬네요.. 제 테스트 환경 코드에서는 아래처럼 되어있었거든요..

<ChattingRoomLayout.Header.Left>
  <div onClick={() => navigate(-1)}>
    <Back />
  </div>
</ChattingRoomLayout.Header.Left>

최신 코드에서는 해당 부분이 전부 수정이 되어있더라구요? 그래서 link를 가져오는 방법만 서버에서 변경한 내용대로 반영하니 모두 해결됐습니다.

질문 혹은 공유 사항 (Optional)

프론트분들이 작업할 내용이 많은 것 같아 큰 변경사항도 없어 제가 그냥 작업했습니다.
테스트는 제 개인 EC2에 배포하고 정상 동작은 확인했어요~

@pricelees pricelees added FE 프론트엔드 관련 이슈입니다. ⚒️ 리팩터링 refactor (기능이 변경되지는 않지만 코드를 수정) labels Oct 17, 2024
Copy link
Contributor

@jaeml06 jaeml06 left a comment

Choose a reason for hiding this comment

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

확인했습니다.

@pricelees pricelees merged commit 463a377 into develop-frontend Oct 18, 2024
1 check passed
@ay-eonii ay-eonii deleted the refactor/#680 branch October 24, 2024 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈입니다. ⚒️ 리팩터링 refactor (기능이 변경되지는 않지만 코드를 수정)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants