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

Fix/#150 프론트가 신고한 에러수정 #156

Merged
merged 13 commits into from
Sep 3, 2024

Conversation

mjj111
Copy link
Member

@mjj111 mjj111 commented Sep 3, 2024

🤨 Motivation

🔑 Key Changes

  • 단순 숫자만 인코딩하여 생기는 문제를 해결했습니다.
  • FCM 엔티티 조회시 사용하는 id값이 MemberId로 사용하던 오류를 수정하였습니다.
  • 매칭 결과 조회시, 이전 매칭회차를 기준으로 조회하도록 하였습니다.
  • application.yml에 있는 내용을 최소화하여 secret.yml에 이동시켰습니다.

🙏 To Reviewers

  • FCM 관련 조회가 안되니까 Event에서 제대로 동작하지 않은 문제였습니다 ㅠ

@mjj111 mjj111 added the fix 오류 관련 태그 label Sep 3, 2024
@mjj111 mjj111 self-assigned this Sep 3, 2024
@mjj111 mjj111 merged commit b7e13d8 into develop Sep 3, 2024
1 check passed
@mjj111 mjj111 mentioned this pull request Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 오류 관련 태그
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant