-
Notifications
You must be signed in to change notification settings - Fork 15
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
[APIS-313] Support ethermint ledger sign(cosmos) #346
Draft
G-Gamja
wants to merge
33
commits into
develop
Choose a base branch
from
feature/APIS-313-support-ethermint-ledger-sign
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…n-chrome-extension into feature/APIS-313-connect-ethermint-ledger
- ethermint펍키를 별도로 선언하지 않고 기존 이더리움 펍키를 활용 - cos_signEIP712 호출 에러에 대한 케이스 정리
- 현재 인젝티브만 기능하고 있음을 확인 todo - 프로토버프로 인코딩 & amino 페이지 내부 리팩토링 필요(중복 코드 제거)
- 이더민트는 compressed된 펍키를 사용하기 때문에 렛저에서 나오는 이더리움 펍키를 그대로 사용하기 어려움 -> 이더리움 펍키 별도로 분리 필요
…n-chrome-extension into feature/APIS-XXX-support-ethermint-ledger-sign
- 렛저 에러 추가 - amino내부에서 doc에 대한 수정 로직 추가 - 이더민트 렛저 사인일때만 (msgs는 더 로우레벨에서 처리중, 이것도 amino페이지에서 다 처리해주면 메시지 타입마다 다 처리해줘야함)
- getRlpTypes에서 tx의 msgs가 모두 동일한 타입일때에메만 리턴을 하도록 로직 변경 - amino 페이지에서 sign버튼을 클릭하면 호출되던 콜백 함수의 결합도를 낮추기 위해 두개의 콜백함수로 분리함
- fee코인을 선택할 때 fee코인 내에서 선택되는게 아닌 tx의 fee.amount필드의 값이 선택되도록 수정했습니다. - 사인 페이지에서 fee코인을 변경할 때 decimals가 변경되지 않는 오류를 수정했습니다.
- Added a new function `toLong` in `big.ts` to handle big numbers using the `Long` library. - Updated `proto.ts` to use the `toLong` function for converting timeout heights and timestamps.
- 일부 변수 변수명 변경 - Long 타입 호출 부분 toLong 유틸 함수 적용
- 체인리스트에 등록되지 않은 코인일지라도 fee코인으로 사용될 수 있기 때문에 무조건적으로 밸런스를 0으로 설정하는 것이 아닌 'balance' 리스폰스의 밸런스 값을 참조한 값이 설정되도록 변경했습니다.
- remove the "ledgerNotSupported" translation strings - add ledger error catching
- Remove unnecessary code in the `entry.tsx` file - Add `useGasMultiplySWR` hook for gas multiplication
…n-chrome-extension into feature/APIS-313-support-ethermint-ledger-sign
…ttps://github.com/cosmostation/cosmostation-chrome-extension into feature/APIS-313-support-ethermint-ledger-sign
…n-chrome-extension into feature/APIS-313-support-ethermint-ledger-sign
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
이더민트 체인의 렛저 계정으로 익스텐션의 기본 기능을 사용할 수 있도록 이더민트 렛저 사인 기능을 추가했습니다.
추가된 기능
변경된 기능
코드 주요 변경점