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.

Sign up for GitHub

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

Jump to bottom

fix(Sentry): Intercepter 적용안되던 것 수정 #439 #440

Merged
merged 96 commits into from
Dec 3, 2022
Merged

fix(Sentry): Intercepter 적용안되던 것 수정 #439 #440

merged 96 commits into from
Dec 3, 2022

Conversation

jooocy
Copy link
Contributor

@jooocy jooocy commented Dec 2, 2022

✅ 풀_리퀘스트 체크리스트

  • PR 제목: [Feature/Fix/Refactor...] 작업 내용 한 줄 요약 (브랜치 이름) #이슈번호
  • commit message 가 적절한지 확인해주세요.
  • 적절한 branch 로 요청했는지 확인해주세요.
  • Assignees, Label 을 붙여주세요.
  • 주의 사항과 관련해 꼭 확인해야 할 사람이 있다면 Reviewer 로 등록해주세요.
  • PR이 승인된 경우 해당 브랜치는 삭제해 주세요!

🔄 변경 사항

  • sentry.intercepter.ts 파일 생성 후 적용했습니다.

📎 변경한 이유

  • Sentry 추적이 안되고 있었습니다.

closes: #Issue Number

👨🏻‍💻 테스트 체크리스트

  • sentry에 log추적하는걸 확인했습니다.

Screenshot from 2022-12-02 10-01-39


📌 변경 및 주의 사항

모든 에러에 메일이 전송됩니다.

jeounpar and others added 30 commits August 26, 2022 12:01
@jeounpar
Merge pull request #222 from 42connected/feature
4dbaa34 
Feature
@rurruur
Merge pull request #225 from 42connected/production
1dcd43d 
배포용
@jeounpar
Merge pull request #227 from 42connected/production
ec44238 
Production
@rurruur
Merge pull request #228 from 42connected/production
5657da9 
Production
@rurruur
Merge pull request #231 from 42connected/production
7dda03a 
Production
@rurruur
Merge pull request #232 from 42connected/production
c45c56a 
Remove: token generator
@jeounpar
Merge pull request #238 from 42connected/production
88e35f1 
Production
@rurruur
Merge pull request #241 from 42connected/production
8806c24 
Production
@jeounpar
Merge pull request #264 from 42connected/feature
815125b 
Feature
@jeounpar
Merge pull request #265 from 42connected/production
0172fcd 
Production
@rurruur
Merge pull request #274 from 42connected/feature
ff49efe 
Feature
@rurruur
Merge pull request #275 from 42connected/production
8e74239 
Production
@rurruur
Merge pull request #278 from 42connected/feature
574f657 
Feature
@rurruur
Merge pull request #282 from 42connected/production
74c00ed 
Production
@rurruur
Update v1.module.ts
910bed3
@kanghyki
Merge pull request #285 from 42connected/feature
15891a3 
Feature
@kanghyki
Merge pull request #287 from 42connected/production
b1a25c6 
Production
@rurruur
Merge pull request #290 from 42connected/feature
39b912c 
Feature
@rurruur
Merge pull request #293 from 42connected/feature
e1c2cdd 
Feature
@rurruur
Merge pull request #291 from 42connected/feature
74453a7 
Feature
@rurruur
Merge pull request #295 from 42connected/feature
8b112da 
Feature
@rurruur
Merge pull request #296 from 42connected/develop
f19c003 
Develop
@rurruur
Merge pull request #299 from 42connected/feature
3b130fe 
Feature
@rurruur
Merge pull request #302 from 42connected/feature
1f1009a 
Feature
@rurruur
Merge pull request #304 from 42connected/feature
a55892a 
Feature
@rurruur
Merge pull request #306 from 42connected/feature
0b7d752 
Feature
@rurruur
Merge pull request #308 from 42connected/feature
b32df83 
Feature
@rurruur
Merge pull request #310 from 42connected/feature
6b99589 
Feature
@rurruur
Merge pull request #313 from 42connected/feature
d96dad9 
Feature
@dkjefilsjl
Merge pull request #315 from 42connected/feature
11fdcc9 
Feature
rurruur and others added 19 commits October 1, 2022 16:12
@rurruur
Merge pull request #391 from 42connected/feature
67648eb 
Feature
@jeounpar
Merge pull request #392 from 42connected/develop
07f05dd 
Develop
@rurruur
Merge pull request #395 from 42connected/feature
98604de 
Feature
@rurruur
Merge pull request #399 from 42connected/feature
d9dcc91 
Feature
@rurruur
Merge pull request #401 from 42connected/feature
ea84fc8 
Feature
@rurruur
Merge pull request #396 from 42connected/develop
34f584d 
Develop
@rurruur
Merge pull request #402 from 42connected/hotfix
e1ec412 
!HOTFIX: 테스트용 라인 삭제...
@rurruur
Merge pull request #404 from 42connected/feature
2a44f97 
Feature
@rurruur
Merge pull request #406 from 42connected/feature
cb1e689 
Feature
@rurruur
Merge pull request #409 from 42connected/feature
7969a1d 
Feature
@rurruur
Merge pull request #407 from 42connected/develop
d7c97dc 
Develop
@rurruur
Merge pull request #411 from 42connected/feature
15acd20 
Feature
@rurruur
Merge pull request #413 from 42connected/feature
5acef6e 
Feature
@rurruur
Merge pull request #414 from 42connected/develop
ab07528 
Develop
@Seojunhwan
feat: apply issue,pull-request template
817793b
@kanghyki
Merge pull request #432 from 42connected/feature
8713c38 
refactor: 멘토링 로그 정렬 순서 변경
@jeounpar
Merge pull request #437 from 42connected/feature
9843a12 
fix(calendar.service): 멘토링 신청/확정 시 신청 시간이 올바르게 처리되지 않던 로직 수정
@jooocy
POL-439 : bug sentry
559d77a
@jooocy
Merge branch 'main' of github.com:42connected/polar-be into POL-439
e725f5a
@jooocy jooocy added the Bug 🚨 Something isn't working label Dec 2, 2022
@jooocy jooocy self-assigned this Dec 2, 2022
@jooocy jooocy linked an issue Dec 2, 2022 that may be closed by this pull request
🚨 [Bug] Sentry #439
Open
3 tasks
@jooocy jooocy merged commit f2a9e5b into feature Dec 3, 2022
@jooocy jooocy deleted the POL-439 branch December 3, 2022 01:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jooocy jooocy

Labels
Bug 🚨 Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

🚨 [Bug] Sentry
7 participants
@jooocy @jeounpar @rurruur @kanghyki @dkjefilsjl @Joo-hyun-Kang @Seojunhwan