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

refactor: 공통 다이얼로그 세분화 #333

Merged
merged 3 commits into from
Jan 16, 2024

Conversation

hoyahozz
Copy link
Member

@hoyahozz hoyahozz commented Jan 13, 2024

Description

  • issue 공통 다이얼로그 슬롯 API 적용 #326
  • 공통으로 사용하는 YDSPopupDialogSlot API 를 적용하였습니다.
    • 두 개의 버튼이 있는 일반적인 YDSPopupDialog
    • 두 개의 버튼 + TextField 가 있는 YDSTextFieldPopupDialog
    • 한 개의 버튼이 있는 YDSSingleButtonPopupDialog

Screen Shots

Screen Shot
YDSPopupDialog

Check List

  • CI/CD 통과여부
  • Develop Mege 시 Squash and Merge 형태로 넣어주세요!
  • 1Approve 이상 Merge
  • Unit Test 작성
  • 연관된 이슈를 PR에 연결해주세요.

@hoyahozz hoyahozz added the 🪄 개선합시다 기능이 기존에 돌아가지만 더 나은 방향으로 개선(ex 리팩토링, 마이그레이션)하려고 할 때 label Jan 13, 2024
Copy link
Member

@toastmeister1 toastmeister1 left a comment

Choose a reason for hiding this comment

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

기존에 YDSPopupDialog를 통해 모든 Dialog를 처리 했었는데 타입별로 나눠주셨군요
한결 깨끗해져서 좋습니다. 고생 많으셨어요!

@toastmeister1
Copy link
Member

공통으로 사용하는 YDSPopupDialog 에 Slot API 를 적용하였습니다.

이 코멘트에 대해서 Slot API를 적용한 부분을 찾지 못했는데
SlotAPI란 하나의 컴포저블에서 파라미터에 또 다른 Composable (ex. content) 를 받아서 유연하게 사용하는것을 뜻해요!
공식문서

예전에 제가 SlotAPI 개념을 활용해서 처리했던 PR인데 참고차 링크 올립니다
DroidKaigi/conference-app-2023#489 (comment)

@toastmeister1 toastmeister1 linked an issue Jan 14, 2024 that may be closed by this pull request
3 tasks
@hoyahozz
Copy link
Member Author

hoyahozz commented Jan 14, 2024

공통으로 사용하는 YDSPopupDialog 에 Slot API 를 적용하였습니다.

이 코멘트에 대해서 Slot API를 적용한 부분을 찾지 못했는데 SlotAPI란 하나의 컴포저블에서 파라미터에 또 다른 Composable (ex. content) 를 받아서 유연하게 사용하는것을 뜻해요! 공식문서

예전에 제가 SlotAPI 개념을 활용해서 처리했던 PR인데 참고차 링크 올립니다 DroidKaigi/conference-app-2023#489 (comment)

아 제가 예전에 저 문서를 보면서 정리를 했었는데, 정리 내용이 완전히 잘못되어 있었군요 '-'

개념 제대로 알려주셔서 감사합니다 ㅎㅎ 👍👍👍

@hoyahozz hoyahozz merged commit 0c33446 into develop Jan 16, 2024
@hoyahozz hoyahozz deleted the refactor/dialog-with-slot-api branch January 16, 2024 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪄 개선합시다 기능이 기존에 돌아가지만 더 나은 방향으로 개선(ex 리팩토링, 마이그레이션)하려고 할 때
Projects
None yet
Development

Successfully merging this pull request may close these issues.

공통 다이얼로그 슬롯 API 적용
2 participants