-
Notifications
You must be signed in to change notification settings - Fork 7
[414] 트랜잭션 임시 저장 #522
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
Open
pio-noncelab
wants to merge
40
commits into
develop
Choose a base branch
from
feat/414-transaction-draft
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.
Open
[414] 트랜잭션 임시 저장 #522
+2,698
−633
Conversation
This file contains hidden or 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
…Draft 구조 수정 임시 저장 트랜잭션 화면구현 임시 저장 트랜잭션 카드구현 - 삭제 기능은 미완성 RealmTransactionDraft 데이터 구조 변경 - 추가 : createdAt, currentUnit, selectedUtxoListJson -삭제 : draftStatus
왼쪽으로 드래그 시 삭제 가능한 기능 구현 화면에 처음 들어왔을 때 '서명 완료' 내역이 없고 '서명 전' 내역은 있을 때 segmentedControl이 자동으로 '서명 전'을 보여줌
…갑 삭제시 해당 임시저장 트랜잭션은 삭제되도록 처리
- 보내기 화면에서 임시저장 트랜잭션 불러오기 기능 구현 - 임시 저장 트랜잭션 화면에서 카드 선택시 보내기 화면으로 이동 및 데이터 입력 구현 - 임시 저장 트랜잭션 삭제 시 삭제완료 팝업 추가 - 임시 저장 트랜잭션 화면에서 카드 드래그를 통한 삭제 - 이미 사용된 UTXO가 포함되어 있을 때 삭제 - 잠금 설정 된 UTXO가 포함되어 있을 때 삭제 - 임시 저장 트랜잭션 중복 체크 제거
- RealmTransactionDraft 필드 추가: totalAmount(서명된 트랜잭션 수량을 임시저장하기 위함)
- 기존 '서명 전' 임시저장 트랜잭션 내용 변경 여부 체크 로직 수정 - '서명 완료' 임시저장 트랜잭션 자동 삭제 기능 추가
- 임시저장 트랜잭션을 불러온 뒤 트랜잭션을 발생시키면 변경사항 유무와 상관없이 무조건 삭제되도록 관련 로직 제거
10 tasks
Collaborator
Collaborator
Collaborator
Collaborator
|
Collaborator
|
보내기 화면 |
…ess Layer 간 DTO 선언 후 적용
…stBottomSheet 위젯 생성 후 적용
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.




변경사항
*CDS 변경이 필요해 임시로 Main 브랜치로 설정해 두었으니, Pull 이후 pub get을 진행해주셔야 합니다.
리뷰 전 필독
1. send_screen_draft.dart
ㄴ send_screen_draft.dart: 👉 send_screen이 너무 길어 임시저장 관련 함수는 여기로 분리해서 part / part of 키워드로 send_screen.dart와 연결시켰습니다.
2. RealmDB에 새로운 테이블을 추가: TransactionDraft
DB Layer보다 상위 Layer에서 사용할 DTO 클래스는 lib/model/wallet/transaction_draft.dart에 정의했습니다.
◦ 서명 전/서명완료 임시 저장 "공통" 항목: id, walletId, recipientJsons, createdAt, feeRate, isMaxMode (임시저장 목록 UI에 보여줘야 해서 서명 완료인 경우에도 저장할 수 밖에 없었습니다.)
◦ 서명 전 임시 저장에만 사용하는 항목: isFeeSubtractedFromSendAmount, bitcoinUnit, selectedUtxoIds (서명 완료 임시 저장의 utxo 검증은 selectedUtxoIds 저장해놓지 않고, 트랜잭션 input에서 utxoId 추출해서 사용했습니다.)
◦ 서명 완료 임시 저장에만 사용하는 항목: txWaitingForSign, signedPsbtBase64Encoded(DTO 클래스에만 있음)
3. 👉signedPsbtBase64Encoded 프로퍼티가 DTO 클래스에만 있는 이유
서명 완료 데이터는 SecureStorage에 별도로 저장하기로 했기 때문입니다.
transaction_draft_repository.dart를 참조해주세요.4. TS/TC ③ 예외 사항 테스트 시 토스트 글자가 하얀색으로 보이는 현상
CDS 업데이트 후 고쳐질 예정입니다.
5. Dropdown 동작
보내기 화면에서 오른쪽 상단 케밥 아이콘 터치 시 나오는 Dropdown 메뉴가 비활성화 상태일 때는 눌러도 Dropdown이 안사라지게 하고 싶었는데, CDS 변경이 필요한 사항인지 의도한대로 되지 않아서 그냥 냅뒀습니다.
#414