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.

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

[fix] issue #270 #285

Merged
merged 7 commits into from
Oct 27, 2024
Merged

[fix] issue #270 #285

merged 7 commits into from
Oct 27, 2024

Conversation

easyhooon
Copy link
Collaborator

@easyhooon easyhooon commented Oct 27, 2024

fix)
kotlin, agp 최신화, type safe navigation 적용시 발생하던 release 빌드에서의 난독화 및 hilt, serialization 관련 문제를 해결하였습니다.

  • 라이브러리 버전 업데이트 kotlin 1.9x -> 2, agp 8.2.2 -> 8.5.2
  • Type Safe Navigation 적용(:core:navigation 모듈은 난독화에서 제외하였습니다. 해당 모듈은 난독화 적용하면 무조건 터지는거 같네요.)
  • 왜 될까요? release 로 빌드해도 잘되네요 뭐가 문제였던거지...
    build-logic 관련해서 수정한 것은 convention package 를 추가해서 거기에 기존 configure 파일들을 위치시켰습니다.(설마 이거 때문인가 하고 패키지 구조 롤백해봤는데, 잘되네여 패키지 구조 문제는 아닙니다.)
    그 밖에는 이것저것 버전 최신화하고, 뚝딱 뚝딱...

진짜 뭐지...?

이전에 kotlin version을 2.0 위로 올리고, agp 8.2.2 였을 때, compose preview 가 동작하지 않는 문제(Android Studio 에 preview tap이 보이지 않음)가 있었는데, 이는 agp version 최신화 하면 해결이 됩니다.(release 빌드가 안되서 이 방법을 선택을 못했었네요 그동안)

기능을 추가하거나 기존의 기능의 버그를 해결한 것은 아니기에 앱을 업데이트할지는 Optional 할 것 같습니다.

feat)

  • 스탬프 화면 새로고침 쓰로틀링 추가
  • 메뉴 화면 관심 축제 텍스트 1줄 제한(글자가 길어지면 텍스트가 줄어들도록)

@easyhooon easyhooon added bug Something isn't working refactor Code restructure for better readability and efficiency labels Oct 27, 2024
@easyhooon easyhooon self-assigned this Oct 27, 2024
@easyhooon easyhooon changed the title Fix/issue 270 [fix] issue #270 Oct 27, 2024
@easyhooon
Copy link
Collaborator Author

일단 제가 체크했을 땐, 문제 없었는데 화면 이동이나 기능에서 문제 있는지 한번 체크 부탁드립니다.

Copy link
Collaborator

@wjdtkdgns777 wjdtkdgns777 left a comment

Choose a reason for hiding this comment

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

LGTM 너무 잘 작동합니다
그.. 근데 왜 될까요 ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

@easyhooon
Copy link
Collaborator Author

easyhooon commented Oct 27, 2024

LGTM 너무 잘 작동합니다
그.. 근데 왜 될까요 ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

버전의 충돌 문제일수도 있고, agp, kotlin, compose, ksp, kotlin-serialization 전부 버전에 민감한 친구들이라... 일단 그정도로 추측해볼 수 있을 것 같아요... 몇 주 동안 골머리를 앓던 문제였는데 어쨌든 해결은 되어서 시원하긴 합니다 ㅎ

@easyhooon easyhooon merged commit 4b16ed0 into develop Oct 27, 2024
1 check passed
@easyhooon easyhooon deleted the fix/issue-270 branch October 27, 2024 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working refactor Code restructure for better readability and efficiency
Projects
None yet
2 participants