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

[2.1.0] 쿠링 구독 대화형 위젯 #212

Merged
merged 17 commits into from
Aug 1, 2024
Merged

Conversation

lgvv
Copy link
Member

@lgvv lgvv commented Jul 24, 2024

브랜치 전략 변화

  • lite 2.0.0 추가
    • 쿠링 라이트를 브랜치입니다. 해당 브랜치에는 기존 라이트처러 실험실적인 부분을 이쪽으로 먼저 머지하고 테플에서 확인 후 dev로 올리거나 폐기하거나 하면 어떨까 싶습니다! (제안)

내용

  • 쿠링 대화형 위젯 앱 (v1- 2023년에 개발되었던 코드를 v2로 이전하였습니다.)
  • appGroup 도입
  • iOS 17이상
  • 인증서 업데이트 된 부분 카톡방에 올려두었습니다.
    • 꼭 확인해보시고 안되는 부분 있으면 제게 말해주세요.
  • 테플도 올려두었습니다. 기능 버그 반드시 체크해주세요!

코드 변경 사항

  • appGroup 들어오면서 migration코드 추가
  • 데이터 저장소를 한곳에 모아두려고 했으나, 기존 작성된 코드의 의존성 문제로 인하여 Model레이어 의존성 관리가 까다로움.
    • 기존 구조에서는 ModelDTO을 범용적으로 사용해서 레이어 나누면 공수가 커서 현재 구조 유지하는 방향으로.

스크린샷

image

@lgvv lgvv changed the base branch from develop/2.0.0 to lite/2.0.0 July 24, 2024 16:45
@lgvv lgvv self-assigned this Jul 24, 2024
Copy link
Member

@sunshiningsoo sunshiningsoo left a comment

Choose a reason for hiding this comment

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

작업 감사합니다! 인증서를 활용한 테스트는 해보지 않았습니다. 다만 dev 브랜치로 가는 것이 아니라 머지 이후에 체크해도 좋을 것 같아요

  • .DS_Store 파일은 삭제하는 것이 좋을 것 같습니다!

@@ -0,0 +1,8 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Member

Choose a reason for hiding this comment

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

s..? 가 어떤 의미 인가요..??

Copy link
Member Author

Choose a reason for hiding this comment

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

의도한 부분은 아니였는데, 인증서 테스트하면서 다른 파일이 들어간 것 같습니다. 요 부분은 체크해보고 의미 없다면 제거할게요!

@lgvv
Copy link
Member Author

lgvv commented Aug 1, 2024

작업 감사합니다! 인증서를 활용한 테스트는 해보지 않았습니다. 다만 dev 브랜치로 가는 것이 아니라 머지 이후에 체크해도 좋을 것 같아요

  • .DS_Store 파일은 삭제하는 것이 좋을 것 같습니다!

.gitignore로 관리하는게 좋을까요 ?

@sunshiningsoo
Copy link
Member

  • .DS_Store 파일은 삭제하는 것이 좋을 것 같습니다!

.gitignore로 관리하는게 좋을까요 ?

이그노어에 있는줄 알았는데 없었군요..!! 이그노어로 관리하는 것 좋은 것 같아요!

@lgvv
Copy link
Member Author

lgvv commented Aug 1, 2024

  • .DS_Store 파일은 삭제하는 것이 좋을 것 같습니다!

.gitignore로 관리하는게 좋을까요 ?

이그노어에 있는줄 알았는데 없었군요..!! 이그노어로 관리하는 것 좋은 것 같아요!

감사합니다! 효원님도 체크주신다면 이건 제가 도서관 작업할 때 같이 챙겨보도록 하겠습니다.

@lgvv
Copy link
Member Author

lgvv commented Aug 1, 2024

@wonniiii 동작성 문제 없다면 lite 브랜치로 바라보고 있어서 머지 눌러주세요!

라이트에 머지되면, 테플로 팀내 QA진행하고, 문제 없으면 Dev로 넣을게요!

Copy link
Member

@wonniiii wonniiii left a comment

Choose a reason for hiding this comment

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

확인했습니다 ! 고생하셨어요

@lgvv lgvv merged commit df30dca into lite/2.0.0 Aug 1, 2024
@lgvv lgvv deleted the feature/widget/appGroup branch August 1, 2024 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants