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

[Feat] #170 월 계산로직 수정 및 에러 핸들링 #176

Merged
merged 5 commits into from
Apr 17, 2023
Merged

[Feat] #170 월 계산로직 수정 및 에러 핸들링 #176

merged 5 commits into from
Apr 17, 2023

Conversation

lsj8706
Copy link
Member

@lsj8706 lsj8706 commented Apr 17, 2023 •
edited
Loading

🌴 PR 요약

🌱 작업한 브랜치

🌱 PR Point

  • SOPT와 N개월째에서 N을 계산하는 로직을 수정했습니다.

📌 참고 사항

  • 사용자가 처음 솝트에 들어온 기수 넙버를 통해 가입일을 구하고 이 가입일과 현재 날짜의 날짜 차이를 이용해 몇 개월째인지를 구했습니다.
  • 현재 MainVC에서 호출하는 api들에 에러 핸들링이 안되어 있는데 이건 어떻게 하면 좋을지 같이 이야기해 봐요!
  • 우선 네트워크 통신하다가 에러가 발생하면 모델에 withError를 true로 바꾸고 이 값을 vc에서 판단하여 Alert뷰를 보여주도록 했습니다.
  • AlertView에서 확인 버튼을 누르면 통신을 재시도 합니다.
  • 서버에서 400번대를 준 경우에는 아마 에러로 인식을 못하는 걸로 알고 있는데 이 때 statusCode를 받아서 에러 처리를 어떻게 할지는 논의가 필요합니다!

📸 스크린샷

기능 스크린샷
사진 image
에러 발생 시
  • 나중에 머지하고 로그인해서 SOPT와 N개월째가 잘 나오는지 확인 한번 부탁드립니다..!

📮 관련 이슈

@lsj8706 lsj8706 added Feat 새로운 기능 구현 세진🤝🏻 labels Apr 17, 2023
@lsj8706 lsj8706 self-assigned this Apr 17, 2023
@lsj8706 lsj8706 changed the title ]Feat] #170 월 계산로직 수정 및 에러 핸들링 [Feat] #170 월 계산로직 수정 및 에러 핸들링 Apr 17, 2023
L-j-h-c
L-j-h-c approved these changes Apr 17, 2023
View reviewed changes
Copy link
Contributor

@L-j-h-c L-j-h-c left a comment

Choose a reason for hiding this comment

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

로직이 훨씬 깔끔해졌네요..멋짐다...!

devxsby
devxsby approved these changes Apr 17, 2023
View reviewed changes
Copy link
Member

@devxsby devxsby left a comment

Choose a reason for hiding this comment

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

전체적으로 에러 핸들링하는 부분 다음에 같이 이야기 나누면 좋을거같아용.
메인피처 말고 다른데서도 써가지구..
고생하셨습니다 🙌🏻

SOPT-iOS/Projects/Features/MainFeature/Sources/MainScene/ViewModel/MainViewModel.swift
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)
Copy link
Member

@devxsby devxsby Apr 17, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

개월수를 서버에서 뿌려주는게 아니였군요.. !!!!
솝트 홈피에는 창립이 2008로 나오긴 하는디 ... 따로 기획한테 안물어보고 작업해도 될가요옹?
특이사항만 있는지 한번만 체크해봐도 좋을거같아여

스크린샷 2023-04-17 22 14 29

Copy link
Member Author

Choose a reason for hiding this comment

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

2008년으로 알고 있기는 했는데 그렇게 하면 기수 계산이 밀려서 2007년으로 두고 했어요!
image
여기에 있는 기수, 날짜를 토대로 식을 세우다 보니 2007년으로 했는데 이 부분은 나중에 서버에서 계산해서 내려주는 걸로 바뀔 것 같아요~!

Copy link
Member

Choose a reason for hiding this comment

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

굿 !! 👍🏻

@pull-request-size pull-request-size bot added size/L and removed size/M labels Apr 17, 2023
@lsj8706 lsj8706 merged commit aa9b004 into sopt-makers:develop Apr 17, 2023
@lsj8706 lsj8706 deleted the feat/#170-월-계산로직-수정-및-에러핸들링 branch April 17, 2023 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@devxsby devxsby devxsby approved these changes

@L-j-h-c L-j-h-c L-j-h-c approved these changes

@elesahich elesahich Awaiting requested review from elesahich

@0inn 0inn Awaiting requested review from 0inn

Assignees

@lsj8706 lsj8706

Labels
Feat 새로운 기능 구현 size/L 세진🤝🏻
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feat] 월 계산 로직 변경 및 에러 핸들링
3 participants
@lsj8706 @L-j-h-c @devxsby