Skip to content

Update NostrConnectMessage display and logs - #1432

Merged
yukibtc merged 2 commits into
masterfrom
nip46-display
Aug 7, 2026
Merged

Update NostrConnectMessage display and logs#1432
yukibtc merged 2 commits into
masterfrom
nip46-display

Conversation

@yukibtc

@yukibtc yukibtc commented Aug 7, 2026

Copy link
Copy Markdown
Member

Description

Update NostrConnectMessage display implementation and reorder NIP-46 message logging in nostr-connect client.

Checklist

  • I followed the contribution guidelines
  • I updated the relevant CHANGELOG.md (if applicable)
  • I understand and can explain all code in this PR

@yukibtc
yukibtc requested a review from TheAwiteb as a code owner August 7, 2026 09:09
@yukibtc yukibtc changed the title Nip46 display Update NostrConnectMessage display and logs Aug 7, 2026
@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
nostr/src/nips/nip46.rs 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Comment thread signer/nostr-connect/src/client.rs Outdated

@TheAwiteb TheAwiteb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

yukibtc added 2 commits August 7, 2026 11:35
…se `Debug` format

Signed-off-by: Yuki Kishimoto <yukikishimoto@protonmail.com>
Signed-off-by: Yuki Kishimoto <yukikishimoto@protonmail.com>
@yukibtc
yukibtc merged commit b3ababb into master Aug 7, 2026
42 checks passed
@yukibtc
yukibtc deleted the nip46-display branch August 7, 2026 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants