Skip to content

[Business][feature] 결제 완료 화면 WebApp bridge 구현#1024

Merged
KYM-P merged 3 commits intodevelop-businessfrom
feature/#1023-orderpage-webapp-bridge
Sep 14, 2025
Merged

[Business][feature] 결제 완료 화면 WebApp bridge 구현#1024
KYM-P merged 3 commits intodevelop-businessfrom
feature/#1023-orderpage-webapp-bridge

Conversation

@KYM-P
Copy link
Copy Markdown
Collaborator

@KYM-P KYM-P commented Sep 14, 2025

PR 개요

이슈 번호: #1023

PR 체크리스트

  • Code convention을 잘 지켰나요?
  • Lint check를 수행하였나요?
  • Assignees를 추가했나요?

작업사항

  • 버그 수정
  • 신규 기능
  • 코드 스타일 수정 (포맷팅 등)
  • 리팩토링 (기능 수정 X, API 수정 X)
  • 기타

작업사항의 상세한 설명

기존 StorePaymentScreenInterface 구현부에

@JavascriptInterface
   fun goToShopDetail(shopId: Int) {
       navigateDetail(shopId)
   }

를 추가로 선언하여
webapp bridge 를 만들었습니다.

추가적으로 JavascriptInterface 의 내부 실행이 background thread 에서 실행될 수 있어서
기존 방식에 따라 (context as Activity).runOnUiThread 을 통해 Ui thread 로 고정했습니다.

논의 사항

현재 누르면 다른 가게로 가는데 이건 이야기 해보겠습니다.

받는 값은 orderable_store_id 인데 제가 store_id 값으로 사용하도록 만들었네요 수정했습니다.

스크린샷

자료(논의 사항 참고)

추가내용

  • develop, sprint 브랜치를 향하고 있습니다
  • production 브랜치를 향하고 있습니다

Add webapp bridge in webs orderpage
Add webapp bridge test class
Feat ktlint
@KYM-P KYM-P self-assigned this Sep 14, 2025
@KYM-P KYM-P requested a review from a team as a code owner September 14, 2025 09:34
@KYM-P KYM-P changed the title Feature/#1023 orderpage webapp bridge [Business][feature] 결제 완료 화면 WebApp bridge 구현 Sep 14, 2025
@github-actions github-actions Bot added the business BCSD business team label Sep 14, 2025
Copy link
Copy Markdown
Member

@kongwoojin kongwoojin left a comment

Choose a reason for hiding this comment

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

고생하셨습니다
몇가지만 확인 해주시고 머지 하시면 될 것 같습니다

import `in`.koreatech.koin.core.webapp.KoinWebAppWebChromeClient
import `in`.koreatech.koin.core.webapp.KoinWebAppWebViewClient

class WebAppBridgeTest {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

작동하는 테스트 코드인가요 👀

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

사실 남겨야하나 고민했는데
webview 를 junit 에서 구동할 수도 없어서
해당 객체를 만들고 실제 activity 에서 복붙해서 테스트 하는 용도로 사용했습니다.
html 코드로 실험해볼까 해서 만들었습니다.

테스트 코드보단 테스트 객체일 뿐이라서... 제거할까요?

}

internal class StorePaymentScreenInterface(
private val navigateDetail: (Int) -> Unit,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

navigateToDetail로 맞춰도 될 것 같습니다

@KYM-P KYM-P merged commit 587fc93 into develop-business Sep 14, 2025
10 of 12 checks passed
@KYM-P KYM-P deleted the feature/#1023-orderpage-webapp-bridge branch September 14, 2025 10:06
@KYM-P KYM-P linked an issue Sep 14, 2025 that may be closed by this pull request
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

business BCSD business team feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Business] 결제 완료 페이지에 WebApp bridge 추가

2 participants