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

#278 [refactor] deprecated된 함수 수정 및 네트워크 에러 분기처리 #279

Merged
merged 34 commits into from
Dec 15, 2023
Merged

#278 [refactor] deprecated된 함수 수정 및 네트워크 에러 분기처리 #279

merged 34 commits into from
Dec 15, 2023

Conversation

2zerozu
Copy link
Contributor

@2zerozu 2zerozu commented Nov 27, 2023 •
edited
Loading

관련 이슈

작업 사진/동영상 (선택)

  • x

작업한 내용

  • serialization 1.9.0 버전업
  • glide -> coil 변경 후 glide 의존성 제거
  • deprecated 된 함수 확장함수로 빼서 처리
  • 강제 업데이트 버전 업데이트하는 로직 수정 @stellar-halo
  • 네트워크 에러 시 분기처리 수정

PR 포인트

image

2zerozu added 23 commits November 28, 2023 05:55
@2zerozu
#278 [build] serialization 버전 업그레이드
d23dfb5
@2zerozu
#278 [build] glide 의존성 제거
8d87928
@2zerozu
#278 [build] okhttp3 의존성 업그레이드 및 bom 제거
90062a2
@2zerozu
#278 [chore] ktlint 적용
8f13494
@2zerozu
#278 [feat] camera required true로 수정
64a4bf9
@2zerozu
#278 [refactor] 이미지 리소스 사용하는 바인딩어댑터 함수명 수정
cc82cf4
@2zerozu
#278 [fix] 불필요한 어노테이션 제거
fd80174
@2zerozu
#278 [fix] DEPRECATION 어노테이션 추가
672e955
@2zerozu
#278 [refactor] parcelable 확장함수 수정
ba0bd6e
@2zerozu
#278 [feat] serializable 확장함수 구현
cc3d8d5
@2zerozu
#278 [refactor] parcelable, serializable 확장함수 적용
ee4226f
@2zerozu
#278 [refactor] binding adapter에 있는 glide를 coil로 대체
a29c458
@2zerozu
#278 [refactor] parcelable 확장함수 적용
358def2
@2zerozu
#278 [refactor] transition 확장함수 적용
df2d48a
@2zerozu
#278 [fix] 바코드 deprecated 된 함수 개선
edcbdc2
@2zerozu
#278 [fix] PickModifyViewModel preListAState, selectItemIdList 접근제한자 수정
e106e7c
@2zerozu
#278 [feat] activityTransition import 추가
10c7547
@2zerozu
#278 [refactor] ktlint
8d266ea
@2zerozu
#278 [feat] getScreenSize 확장함수 구현
fce48a3
@2zerozu
#278 [feat] getScreenSize 확장함수 적용
6022103
@2zerozu
#278 [refactor] 권한 변수명 수정
1324263
@2zerozu
#278 [fix] BarcodeAnalyser analyze 함수 SuppressLint를 OptIn으로 수정
3fe1591
@2zerozu
#278 [refactor] BarcodeAnalyzer 클래스명 수정
c1b94a3
@2zerozu 2zerozu changed the title Feature/#278 refactor deprecated function #278 [refactor] deprecated된 함수 수정 Nov 27, 2023
@2zerozu 2zerozu changed the title #278 [refactor] deprecated된 함수 수정 #278 #280 [refactor] deprecated된 함수 수정 Dec 13, 2023
@2zerozu 2zerozu self-assigned this Dec 13, 2023
@2zerozu 2zerozu added fix 버그 개선 Pull Request🔥 풀리퀘 날림! refactor 내부 로직은 변경 하지 않고 기존의 코드를 개선하는 리팩토링 시 영주🐼 영주가 작업함! labels Dec 13, 2023
@2zerozu 2zerozu changed the title #278 #280 [refactor] deprecated된 함수 수정 #278 [refactor] deprecated된 함수 수정 Dec 13, 2023
@2zerozu 2zerozu changed the title #278 [refactor] deprecated된 함수 수정 #278 [refactor] deprecated된 함수 수정 및 네트워크 에러 분기처리 Dec 13, 2023
@2zerozu 2zerozu requested a review from a team December 13, 2023 13:22
stellar-halo
stellar-halo approved these changes Dec 13, 2023
View reviewed changes
Copy link
Member

@stellar-halo stellar-halo left a comment

Choose a reason for hiding this comment

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

아휴 이놈의 버전,, 너무 고생했어요👍🏻👍🏻

app/src/main/java/com/sopt/peekabookaos/domain/entity/Version.kt
val androidForceVersion: String = "",
val versionText: String = ""
) : Parcelable
fun greaterThan(other: Version): Boolean = other.major < major || other.minor < minor
Copy link
Member

Choose a reason for hiding this comment

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

boolean 을 반환하는 경우는 isGreaterThan처럼 보통 쓴다고 알고 있는데 혹시 어떤가요!?

Copy link
Contributor Author

@2zerozu 2zerozu Dec 13, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

boolean 을 반환하는 경우는 isGreaterThan처럼 보통 쓴다고 알고 있는데 혹시 어떤가요!?

모든 boolean에 is 접두사를 붙일 필요는 없어요!
greaterThan이 ~보다 크다 라는 뜻의 함수로 쓰인 거라 greaterThan(cutrentVersion)이라면 현재 버전보다 높다(false라면 낮다)이기 때문에 boolean이라 예상 가능해보입니다
isGreaterThan이라면 is가 불용어가 될 거 같아용

app/src/main/java/com/sopt/peekabookaos/presentation/forceUpdate/ForcedUpdateActivity.kt
import com.sopt.peekabookaos.util.extensions.initBackPressedCallback
import com.sopt.peekabookaos.util.extensions.setSingleOnClickListener

class ForcedUpdateActivity :
Copy link
Member

Choose a reason for hiding this comment

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

아예 액티비티를 만드셨군요..!!

Copy link
Contributor Author

@2zerozu 2zerozu Dec 13, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

아예 액티비티를 만드셨군요..!!

이거 액티비티명을 바꿨더니 새로만든걸로 뜨네 .. 이름만 바꿨어융
image

hajeong67
hajeong67 approved these changes Dec 15, 2023
View reviewed changes
Copy link
Contributor

@hajeong67 hajeong67 left a comment

Choose a reason for hiding this comment

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

함수 수정 확인했습니다! 바쁜데도 고생많아요

app/src/main/java/com/sopt/peekabookaos/data/service/ForcedUpdateService.kt
import retrofit2.http.GET

interface ForcedUpdateService {
@GET("user/v1/version")
Copy link
Contributor

Choose a reason for hiding this comment

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

👍👍👍👍

@2zerozu 2zerozu merged commit fb19397 into develop Dec 15, 2023
1 check passed
@2zerozu 2zerozu deleted the feature/#278-refactor-deprecated-function branch December 15, 2023 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hajeong67 hajeong67 hajeong67 approved these changes

@stellar-halo stellar-halo stellar-halo approved these changes

Assignees

@2zerozu 2zerozu

Labels
fix 버그 개선 Pull Request🔥 풀리퀘 날림! refactor 내부 로직은 변경 하지 않고 기존의 코드를 개선하는 리팩토링 시 영주🐼 영주가 작업함!
Projects
None yet
Milestone
피카북 1차 릴리즈 이슈 대응
3 participants
@2zerozu @stellar-halo @hajeong67