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

[CAT-161] Feature: token interceptor를 포함한 API 모듈 수정 #18

Merged
merged 4 commits into from
Aug 7, 2024
Merged

[CAT-161] Feature: token interceptor를 포함한 API 모듈 수정 #18

merged 4 commits into from
Aug 7, 2024

Conversation

Jihyun247
Copy link
Collaborator

[#161] token interceptor를 포함한 API 모듈 수정

무엇에 관한 PR 인가요? 🙋

API 모듈 구조 변경 및 token interceptor 구현

🌱 PR Point

  • token header를 넣어주는 adapt() 와 token을 갱신해주는 retry()를 가지고 있는 TokenInterceptor
  • 두번 실패 후 세번 시도 시 timeout이 뜨게 했어욥 ..

@Jihyun247 Jihyun247 added the enhancement New feature or request label Aug 7, 2024
@Jihyun247 Jihyun247 requested a review from devMinseok August 7, 2024 15:40
@Jihyun247 Jihyun247 self-assigned this Aug 7, 2024
@linear Linear
Copy link

linear bot commented Aug 7, 2024

CAT-161 [iOS] API 모듈 구조 변경

devMinseok
devMinseok approved these changes Aug 7, 2024
View reviewed changes
Copy link
Member

@devMinseok devMinseok left a comment

Choose a reason for hiding this comment

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

고생했어~

@Jihyun247 Jihyun247 merged commit 9d338cc into develop Aug 7, 2024
1 check passed
@Jihyun247 Jihyun247 deleted the feature/CAT-161 branch August 7, 2024 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@devMinseok devMinseok devMinseok approved these changes

Assignees

@Jihyun247 Jihyun247

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Jihyun247 @devMinseok