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

localStorage 에 블록체인 지갑 목록 저장하기 #1

Closed
junhoyeo opened this issue May 18, 2022 · 4 comments · Fixed by #8
Closed

localStorage 에 블록체인 지갑 목록 저장하기 #1

junhoyeo opened this issue May 18, 2022 · 4 comments · Fixed by #8
Assignees

Comments

@junhoyeo
Copy link
Member

  • 현재는 @dashboard/coreconfig 에 하드코딩 되어 있음
  • 클라이언트에서 주입 가능하도록 만들어야 함
@citrusinesis
Copy link
Contributor

citrusinesis commented May 19, 2022

localStorage에 저장하는 로직까지는 완성.

ToDo

  • form에서 네트워크 종류 multiple select구현
  • 에러 핸들링

고민

  • Wallet type state의 요소인 type부분에서 에디터가 지속적인 Warning을 띄움. (string cannot assign to "'solana'") 이를 해결하기 위해 config.ts에서 빈 Wallet객체를 만들어 radio버튼에 따라 그 객체를 복사 해 오는것은?

@junhoyeo
Copy link
Member Author

string cannot assign to "'solana'"

  • @harugatto 지금까지 작업하신 내용으로 PR을 열어보는 건 어떨까요? 코드 리뷰를 해드릴 수 있을 것 같아요.
  • as로 타입을 강제로 바꾸거나, as const 를 사용해 정적으로 타이핑된 리스트를 만든 다음 radio 를 사용해서 해결할 수도 있겠습니다!

@junhoyeo
Copy link
Member Author

@harugatto 요거 제가 작업하신 브랜치에서 커밋 추가해볼게요! 수고하셨어요 ❤️

@junhoyeo
Copy link
Member Author

+ 앞으로는 브랜치 만드실 때 [username]/[feature-description] 형식을 써보는 건 어떠세요? ex> harugatto/store-wallets-in-local-storage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants