Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tokenCheck 메소드 이름보단 checkToken 이 더 좋을 것 같아 수정하였습니다
AuthController 란 이후 자동 로그인 구현에 있어 미리 구현해본 객체입니다. 아직까진 사용을 안하니 눈여겨볼 필요는 없을 것 같습니다 !!
한 번 재발급을 신청한 인터셉터는 플레그를 세웁니다. 이후 플레그가 있으면 토큰을 재발급을 하지 않게 하여, 토큰이 반복적으로 재발급되는 상황을 막았습니다.
Open
|
Visit the preview URL for this PR (updated for commit cfe84d2): https://taskweaver-front--pr101-100-apiresponse-m7pwkm0f.web.app (expires Sun, 09 Jun 2024 14:11:32 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 37daec54a48dbfe281302dc48a4e028c965a31c2 |
jjoonleo
reviewed
May 18, 2024
| import 'package:front/features/user/data/data_source/local_data_source/user_local_data_source.dart'; | ||
| import 'package:front/features/user/data/data_source/remote_data_source/user_remote_data_source.dart'; | ||
| import 'package:front/features/user/data/models/change_password.dart'; | ||
| import 'package:front/features/user/data/models/sign_in_request.dart'; |
There was a problem hiding this comment.
모델파일 이름도 다른 파일들과 마찬가지로 _model로 끝나고 class이름도 Model로 끝나는 것이 좋을 것 같습니다.
There was a problem hiding this comment.
아 넵넵 알겠습니다 ! 수정하겠습니다 ! 감사합니다 !
jjoonleo
reviewed
May 18, 2024
There was a problem hiding this comment.
이부분은 혹시 어떤 역할을 하는 뷰모델 인가요?
viewmodel은 파일이름과 class이름을 viewmodel로 바꾸는 것이 좋을 것 같습니다.
There was a problem hiding this comment.
아 이 부분은
main Home 에서 팀 선택 시, 팀의 정보를 받아오는 provider 입니다.
말씀하신대로 provider 보단 viewModel 이름으로 바꾸는 게 좋을 것 같습니다 !
jjoonleo
approved these changes
May 27, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[fix: api 삭제]
[fix: user 및 team 데이터 소스 수정]
[fix: user 및 team 레파지토리 수정]
[fix: user 및 team usecase 수정]
[fix: viewModel 수정]
[fix: di 및 locator 수정]
[fix: 로그인 페이지 수정]
[fix: 메소드 이름 수정]
[fix: AuthContoller 수정]
[fix: token 재발급이 반복적으로 되는 에러 해결]
[fix: 파일 이름 수정으로 인한 import 문구 수정]