don't render nostr:npub as a link #528
No reviewers
Labels
No Label
1000k
100k
10k
200k
20k
500k
50k
5k
75k
backend
blocked:design
bug
dependencies
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
scope:intl
scope:nip
scope:query_tracing
scope:ux
wontfix
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Kieran/snort#528
Loading…
Reference in New Issue
No description provided.
Delete Branch "fix-nostr-npub-broken-links"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
resolves https://github.com/v0l/snort/issues/501
People often type the literal text
nostr:npub
when talking about nostr URIs. That shouldn't be converted to a link.Deploying with Cloudflare Pages
969284e
View logs
There is also an issue where parsing an invalid TLV will break the note, for example trying to preview note with content
nostr:nevent1xxx
Fixing this should also fix the invalid
nostr:npub
I looked into it and that seems to be a separate issue, but I can try and fix both with this PR.
I think I was able to fix both issues.
@v0l fixed the logic to work for all nip-19 types and added unit tests.