feat: 서랍장 서비스 등록 페이지 네이티브 웹뷰 연결 #216
Conversation
2wndrhs
requested review from
nijuy,
Hanna922,
seocylucky,
intersoom,
ssolfa,
fecapark,
jonique98,
owl1753 and
minai621
as code owners
| @@ -40,6 +57,22 @@ export const Register = () => { | |||
| } | |||
| }; | |||
|
|
|||
| useEffect(() => { | |||
| const getAccessTokenFromNative = () => { | |||
There was a problem hiding this comment.
앞으로 해당 객체들을 사용할 때 마다 if문으로 undefined 체크를 해야 할 것 같습니다.
initialize 할 수 있는 기능을 추가해야 할 것 같은데 아래처럼 클래스 두고 인스턴스 생성해서 쓰는건 별로일까요?
class NativeAPI {
constructor(iosHandler, androidHandler, api) {
this.initialize(iosHandler, androidHandler);
}
initialize(iosHandler, androidHandler) {
if (!iosHandler || !androidHandler) {
throw new Error("No valid handler found for Native API.");
}
this.handler = iosHandler;
this.handler = androidHandler;
}
// awesome logics..
}
There was a problem hiding this comment.
어썸 로직이네요................................
There was a problem hiding this comment.
고민을 해봤는데 해당 코드는 웹뷰로 접속하는 페이지가 많을 때 유용할 것 같은데 숨쉴때는 서랍장 서비스 등록 페이지만 웹뷰로 접속하는 상황이라 현재 상황에서는 지금 코드를 유지해도 될 것 같아요
웹뷰로 접속하는 페이지가 많아지고 반복되는 로직을 제거해야할 필요성을 느낄 때 제안해주신 코드를 도입하면 좋지 않을까 싶네요..!
어썸 코드 감사합니다
| }; | ||
|
|
||
| window.setAccessToken = (accessToken: string) => { | ||
| api.setAccessToken(accessToken, Date.now() + 5 * 60 * 1000); |
There was a problem hiding this comment.
모바일 토큰의 브라우저 상 유효시간을 받아온 시각 + 5분으로 설정한 거는 쭌이 임의로 정한건가요? 혹시 타자가 느려서 웹뷰 접속 후 5분 지나면........ 어쩌지 하는 걱정이 있어가지고
유효 시간은 넉넉하게 해두고 프로덕트 등록 mutate 완료 후에 토큰 삭제하는 건 별로일까요?!?
There was a problem hiding this comment.
네이티브 쪽에서 accessToken만 받아오는거라 제가 임의로 5분으로 설정했는데 생각해보니 조금 짧은 감이 있네요..
1시간으로 넉넉히 잡아놓겠습니다 -> 96c3bde
유저가 서비스 등록 후에 바로 다른 서비스를 등록할 수 있으니 토큰은 살려두는게 나을 것 같아요..!
맞네요 서비스 등록 성공하면 페이지 이동시키는 코드 추가해놓겠습니다..! -> b5379a5 웹뷰에서는 서비스 등록 성공시 웹뷰 안에서 페이지 이동을 시키는게 어색할 것 같아서 다시 네이티브 쪽으로 성공 여부를 알려줘서 네이티브 쪽에서 처리하는게 자연스러울 것 같네요 -> 이 부분은 안드, ios 팀이랑 얘기해보겠습니다 |
저도 웹뷰 연결이 처음이라 지피티씨 도움 받아서 구현했습니다🤗 |
2wndrhs
force-pushed
the
feat/#184-service-register-webview
branch
from
b5379a5
to
f941fef
Compare
2wndrhs
force-pushed
the
feat/#184-service-register-webview
branch
from
7a174ae
to
8e5e87a
Compare
1️⃣ 어떤 작업을 했나요? (Summary)
기존 코드에 영향을 미치지 않는 변경사항
drawer/pages/Register.tsxaccessToken을 받아오는 기능을 구현하였습니다.2️⃣ 알아두시면 좋아요!
Android 웹뷰에서 서랍장 서비스 등록 페이지에 접속
accessToken을 가져와 웹뷰에 정의된 자바스크립트 함수인window.setAccessToken()을 호출setAccessToken함수는 네이티브로부터accessToken을 받아api싱글톤 객체에 등록 -> 추후 API 요청에 이용ios 웹뷰에서 서랍장 서비스 등록 페이지에 접속
accessToken을 요청(window.webkit.messageHandlers.ios.postMessage("getAccessToken"))accessToken을 가져와 웹뷰에 정의된 자바스크립트 함수인window.setAccessToken()을 호출setAccessToken함수는 네이티브로부터accessToken을 받아api싱글톤 객체에 등록 -> 추후 API 요청에 이용3️⃣ 추후 작업
4️⃣ 체크리스트 (Checklist)
main브랜치의 최신 코드를pull받았나요?