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

Refac/UI #58

Merged

mqjinwon merged 2 commits into main from refac/ui

Nov 6, 2024

Collaborator

mqjinwon commented Nov 6, 2024

설명

UI 를 base_extension에서 구현하는 것이 아닌, 상속받은 주체 extension에서 구현을 하도록 만듬

_build_ui를 @AbstractMethod 로 만들어서 상속이 필수적으로 이루어지도록 만듬

변경 유형

리팩토링 (기능적 변경 없음, API 변경 없음)

체크리스트:

내 코드가 프로젝트의 코드 스타일 가이드라인을 따릅니다
내가 내 코드를 자체 리뷰했습니다
내 변경사항으로 인해 새로운 경고가 발생하지 않습니다
단위 테스트를 추가했습니다
기존 단위 테스트가 모두 통과합니다
[] 필요한 경우 해당 문서를 변경했습니다
내 변경사항에 대한 종속성 업데이트가 포함되어 있습니다

mqjinwon added 2 commits

November 6, 2024 14:26


          refac: ui part를 상속받아 구현하도록 변경

98faa82


          code formatting

d51bf73

mqjinwon added the enhancement label

mqjinwon self-assigned this

mqjinwon merged commit 607492b into main

1 check passed

mqjinwon deleted the refac/ui branch

November 6, 2024 14:29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels