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

[Refector] common-lib 추가 #8

Merged
merged 4 commits into from Sep 19, 2022
Merged

[Refector] common-lib 추가 #8

merged 4 commits into from Sep 19, 2022

Conversation

nodobi
Copy link
Contributor

@nodobi nodobi commented Sep 19, 2022

처음 멀티 모듈 구현했을 때 만들었다가 없앤 common-lib를 다시 만들었어요

  • Viewer, Controller 내부 EuPICodeEnum 파일 삭제
  • common-lib 에 EuPICodeEnum 파일 추가
  • 사용하지 않는 import, 변수 삭제
  • 로그 내용 수정

@phj0407
Copy link
Collaborator

phj0407 commented Sep 19, 2022

확실히 Common_lib 부활시키니까 깔끔해졌네요ㅋㅋ 로드 정리도 좋습니다! 핸드폰-에뮬로도 테스트 되는지 한 번만 확인해보고 머지하면 될 것 같아요!

@nodobi
Copy link
Contributor Author

nodobi commented Sep 19, 2022

코드 한번 전체적으로 다시 확인해봤는데, 권한 확인하는 부분에서 제가 엉뚱한 권한을 확인하고 있더라구요.. ㅋㅋㅋㅋ
그 부분 수정했고, 다른 안쓰는 코드들도 같이 제거했습니다!

@phj0407 phj0407 merged commit 28d2fae into euphony-io:main Sep 19, 2022
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.

None yet

2 participants