[Fix] #399 - deprecated API 반영 및 가입 일자 계산 로직 수정 #401
Merged
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
- 마이페이지에서 현재 알림 수신 여부를 표시하는 기능이나 현재는 바로 setting으로 이동하기 때문에 불필요
- 현재 signup은 네이티브앱에서 진행하지 않음
ConfigService, ConfigAPI, 관련 Model, Entity제거 repository, usecase 내 getServiceState 관련 코드 제거
- dateComponent에 9시간을 더해 한국 일자로 수정 - 동아리 시작 일자를 2일로 계산하여 안드로이드 계산법과 맞춤
|
yungu0010
changed the title
yungu0010
changed the title
dlwogus0128
approved these changes
Oct 15, 2024
| @@ -57,7 +57,7 @@ extension DailySoptuneResultViewModel { | |||
| let output = Output() | |||
| self.bindOutput(output: output, cancelBag: cancelBag) | |||
|
|
|||
| input.viewWillAppear | |||
| input.viewDidLoad | |||
| @@ -339,7 +331,7 @@ extension MainViewModel { | |||
| let yearDifference = history / 2 | |||
| let month = (history % 2 == 0) ? 3 : 9 // 짝수 기수는 3월, 홀수 기수는 9월 시작 | |||
| // 1기를 2007년으로 계산 | |||
| return Date.from(year: yearDifference + 2007, month: month, day: 1) | |||
| return Date.from(year: yearDifference + 2007, month: month, day: 2) | |||
| } | |||
meltsplit
reviewed
Oct 16, 2024
Comment on lines
26
to
28
| return calendar.date(from: dateComponents) ?? nil | ||
|
|
||
| let usaDate = calendar.date(from: dateComponents) | ||
| return usaDate!.addingTimeInterval(10 * 60 * 60) |
There was a problem hiding this comment.
-
9시간이 아닌 10시간을 더하신 이유가 궁금해요!
-
TimeZone.current로 설정하면 한국날짜(UTC +9)로 지정되는 것으로 알고 있는데,
해당 데이터에 추가적으로 9시간 혹은 10시간을 더하면 (UTC +18)이 되는 건 아니겠죠,,?
There was a problem hiding this comment.
- 10시간은 제가 이것저것 해보다가 그대로 올린 것 같습니다ㅎㅎ 수정하겠습니다!
- calendar.date 함수의 반환값이 Date 객체인데, Date 객체는 항상 UTC 기준으로 생성된다고 해요. Date의 시간대를 변경하려면 DateFormatter를 사용해야하는데 DateFormatter를 사용하는 것보다 시간을 더해주는 것이 오버헤드가 더 적어 기존 방법을 유지하는 것을 좋을 듯 합니다 !
덕분에 새로이 알고 갑니다!_!👍
This comment was marked as duplicate.
This comment was marked as duplicate.
Sorry, something went wrong.
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.
🌴 PR 요약
🌱 작업한 브랜치
🌱 PR Point
사용하지 않는 API 제거
GET /notification/{notificationId}사용하지 않음 → 삭제GET /user/nickname/{nickname}제거된 기능 → 삭제GET /user/nickname제거된 기능 → 삭제GET /s3/mission→/s3/stamp로 변경GET /config/availability삭제/config/availability는 앱 서비스의 심사 여부를 true, false로 내려주는 역할로 추정
확인해본 결과 프로젝트 로직에서도 false인 경우 따로 처리해주지 않고 서버측에서도 false를 내려준 적이 없다고 하여 제거했습니다.
GET /user/opt-in→ 이전 버전에서는 앱 알림 수신 동의 여부를 마이페이지에서 나타낼 때 사용한 것으로 추정
현재는 수신 동의 여부를 보여주지 않고 바로 설정 앱으로 이동하기 때문에 사용하지 않아 삭제했습니다.
홈화면 동아리 시작 개월수 설정 변경
안드로이드와 iOS에서 표시되는 개월수가 상이해 iOS가 AOS 기준으로 싱크를 맞추고, 추후 서버 API를 통해 값을 받는 방식으로 변경하기로 했습니다!
문제 원인은
calendar.dateComponents가 안드로이드와 계산 방식이 다르기 때문이었어요.그래서 동아리 시작날짜는 1일이 아닌 2일로 지정하여 AOS와 싱크를 맞췄습니다.
추가로 시작 일자 계산이 미국시 기준으로 되어있어 9시간을 추가해 한국일자로 변경하였습니다.
📌 참고 사항
📮 관련 이슈