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.

Sign up for GitHub

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

Jump to bottom

[draft] contact list #4434

Draft
wants to merge 30 commits into
base: main
Choose a base branch
from
Draft

[draft] contact list #4434

wants to merge 30 commits into from

Conversation

obrusvit
Copy link
Contributor

Ignore.
Draft PR for the purpose of Suite.

@obrusvit
feat(contacts): add protobuf definitions
3e9eb07
@obrusvit
feat(contacts): sign receive address with
beb2200 
- bitcoin receiving address is signed with private key of the first
testnet bitcoin address
- signature is passed in `Address` response
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 10, 2024 •
edited
Loading

core UI changes device test click test persistence test
T2T1 Model T test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
T3B1 Safe 3 test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
T3T1 Safe 5 test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
All main(screens)

@mroz22 mroz22 mentioned this pull request Dec 10, 2024
Draft
martykan and others added 25 commits December 10, 2024 14:57
@martykan
feat(signing): show and verify label when signing
ec36da2
@martykan
test(contacts): signing flow device test
a8344aa
@martykan
fixup! feat(signing): show and verify label when signing
e6b05a7
@obrusvit
feat(contacts): implement layout for TXOUT
10269d7 
- SwipeFlow for TXOUT confirmation for labeled contact
@obrusvit
chore: regenerate proto
97ab095
@obrusvit
feat(contacts): put big capital letters to mercury
9b43f13 
- DO NOT USE IN PRODUCTION
@obrusvit
refactor(contacts): cleanup getaddress
66e2ada
@obrusvit
feat(contacts): improve confirm_output_contact
951be5a 
- show correct titles and subtitles
- remove key labels
@obrusvit
feat(contacts): new contact confirmation
7c375bd
@obrusvit
feat(contacts): improve confirm_new_contact
4ca2e71
@martykan
fixup! feat(contacts): sign receive address with
010fdda
@ibz @obrusvit
feat(core): add nostr
c915904
@obrusvit
feat(contacts): sign btc address with nostr
38977ac
@obrusvit
fixup! feat(contacts): improve confirm_new_contact
4e50e64
@obrusvit
fix(contacts): remove unused nostr proto fields
bfc444f
@obrusvit
feat(contacts): new contact UI using nostr event
6151524
@obrusvit
ci: allow arm build
38936a1 
DO NOT USE IN PRODUCTION
@obrusvit
feat(nostr): extend GetPublicKey with show_display
cb913ad
@obrusvit
ci: allow arm build part 2
d758453
@obrusvit
feat(nostr): UI for get_pubkey
baa9f37
@obrusvit
feat(contacts): switch label approver to nostr
15f7d1d 
WIP: there seems to be a problem
@ibz
fixup! feat(contacts): sign btc address with nostr
2ce2443
@martykan
feat(trezorctl): view signature for btc get–address
bb62045
@martykan
fix(contacts): nostr pubkey verify issue
f674833
@ibz
fixup! feat(contacts): sign btc address with nostr
9a3f5d0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matejcik matejcik Awaiting requested review from matejcik matejcik will be requested when the pull request is marked ready for review matejcik is a code owner

@prusnak prusnak Awaiting requested review from prusnak prusnak will be requested when the pull request is marked ready for review prusnak is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
Firmware
Status: 🔎 Needs review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@obrusvit @martykan @ibz