-
Notifications
You must be signed in to change notification settings - Fork 3
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
[Update] Reduce StoreActivity listening feature #34
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다!! 머지되면 실기기로 테스트 해볼게요!
@phj0407 @SeonJK 사실 EuRxManager나 EuTxManager는 Singleton 처럼 사용되는 걸 의도하긴 했으나 나중에 이러한 코드 방지를 위한 EuTxManager나 EuRxManager에 Singleton 적용을 여러분이 해주시면 좋겠다는 생각이 드네요. SIngleton 적용도 Dashboard Core에 추가 완료하였습니다! mEuPIRxManagerMenu = EuRxManager(EuOption.ModeType.EUPI)
mEuPIRxManagerMenu.setOnWaveKeyDown(RequestCodeEnum.MENU_REQUEST.code.toInt()) {
Toast.makeText(this, "Menu request detected", Toast.LENGTH_SHORT).show()
Log.d(TAG, "mEuPIRxManagerMenu - Receive menu request.")
}
mEuPIRxManagerOrder = EuRxManager(EuOption.ModeType.EUPI)
mEuPIRxManagerOrder.setOnWaveKeyDown(RequestCodeEnum.ORDER_REQUEST.code.toInt()) {
Toast.makeText(this, "Order request detected", Toast.LENGTH_SHORT).show()
Log.d(TAG, "mEuPIRxManagerOrder - Receive order request.")
} 위 코드는 아래처럼 쓰는걸 권장합니다 :) mEuPIRxManager = EuRxManager(EuOption.ModeType.EUPI)
mEuPIRxManager.setOnWaveKeyDown(RequestCodeEnum.MENU_REQUEST.code.toInt()) {
Toast.makeText(this, "Menu request detected", Toast.LENGTH_SHORT).show()
Log.d(TAG, "mEuPIRxManagerMenu - Receive menu request.")
}
mEuPIRxManager.setOnWaveKeyDown(RequestCodeEnum.ORDER_REQUEST.code.toInt()) {
Toast.makeText(this, "Order request detected", Toast.LENGTH_SHORT).show()
Log.d(TAG, "mEuPIRxManagerOrder - Receive order request.")
} |
아하, 굳이 오브젝트를 두 개 생성하지 않아도 되는 거였군요ㅎㅎ! 수정했습니다~~ 나머지는 이대로 머지해도 괜찮을까요? |
@phj0407 넵! |
#33 에 올렸던 것 처럼 EuPI 를 이용해서 메뉴/주문 요청 구분해서 메시지 띄우는 기능까지만 정리했습니다. 기기가 하나 뿐이라 테스트는 아직 못 해봤네요ㅠㅠ 시간 되실 때 확인 부탁드리겠습니다!