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] log 2개 보이는 문제 해결 + [add] 사용자 fcm 토큰 수정 api 추가 #65

Merged
merged 4 commits into from
Feb 1, 2022

Conversation

hyejungg
Copy link
Member

@hyejungg hyejungg commented Feb 1, 2022

What is this PR? 🔍

log가 2개씩 뜨는 문제 해결

Key Changes 🔑

  1. log가 2개씩 뜨는 문제
    • createLogger 부분에서 에러 발생 시 바로바로 뜨도록 console에 추가하는 옵션을 추가했으나 이 문제로 log가 2개씩 뜨는 문제를 뒤늦게 발견하여 삭제하였습니다.
    • log는 현재 debug는 log 파일에서만 볼 수 있고 개발 시 콘솔에서 볼 수 없는 문제, 자세하게 err내용을 보여주지 않아 추측해야하는 문제로 인해서 더 자세하게 볼 수 있도록 개선할 예정입니다. (log 개선 #66)
//config/winston.js
new winston.transports.Console({ handleExceptions: true }),

To Reviewers 📢

  • 원래 firebase 아이디/비밀번호 구현 후 미리 fcm 토큰 주석 처리 한 커밋(사용자 API URI 개선 및 예외처리 추가 #43)을 같이 해당 브랜치로 업로드 할 예정으로 브랜치를 합쳐두었는데 hotfix 로 인해 해당 커밋도 올라갔습니다 .. 양해 부탁드립니다.

@hyejungg hyejungg added bug bug 및 hotfix 수정 feat 새 기능 추가 mod 코드 수정 labels Feb 1, 2022
@hyejungg hyejungg requested a review from youngjinc February 1, 2022 14:22
@hyejungg hyejungg self-assigned this Feb 1, 2022
@hyejungg hyejungg merged commit 296e4ce into hyeeyoung:dev Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bug 및 hotfix 수정 feat 새 기능 추가 mod 코드 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant