-
Notifications
You must be signed in to change notification settings - Fork 578
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
Relationship status #761
Draft
vitorpamplona
wants to merge
32
commits into
nostr-protocol:master
Choose a base branch
from
vitorpamplona:relationship-status
base: master
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.
Draft
Relationship status #761
Changes from 1 commit
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
a3292f4
First Draft
vitorpamplona b7aa64d
Quick adjustments
vitorpamplona 0006b13
adds privately
vitorpamplona 5cc0c93
correct typo
sudocarlos fe00ddd
Merge pull request #1 from sudocarlos/patch-1
vitorpamplona dc4af5d
Refines proposal to make relationships private.
vitorpamplona bd1755f
Improves formatting
vitorpamplona 73a2aab
Optional n tag
vitorpamplona 225be8a
correcting the example
vitorpamplona 0f4edef
Switch to random UUID for better security.
vitorpamplona c31a504
Updating NIP-81 to add private and public statuses
vitorpamplona 6868e02
formatting
vitorpamplona ee6d4e6
enhancing the example with private ntags
vitorpamplona 1bcbd31
wording
vitorpamplona df362d1
formatting
vitorpamplona 873569c
formatting
vitorpamplona fbb5aba
Merge remote-tracking branch 'upstream/master' into relationship-status
vitorpamplona 1e7da2b
Moves to 10008 kind for name maps
vitorpamplona 3ff65d1
Adds edge cases
vitorpamplona a401682
renames nicknames to petnames
vitorpamplona 61898c0
Simplifying statuses
vitorpamplona 4482a41
Creating two event kinds.
vitorpamplona a3a6784
Update 81.md
vitorpamplona 08d8b1e
Update 81.md
vitorpamplona a5ff486
Update 81.md
vitorpamplona 89e8124
Merge remote-tracking branch 'upstream/master' into relationship-status
vitorpamplona bae459a
Updates relationship status to use event sets.
vitorpamplona a8c573f
adds public relationship status to the readme
vitorpamplona 4e0b5fa
Aligning the Private Relationship status Kind with the other PRs
vitorpamplona 3ba8526
Fixing kind numbers
vitorpamplona 8cb2ebc
Merge remote-tracking branch 'upstream/master' into relationship-status
vitorpamplona 8dafaf8
moves event set names to the right position on readme
vitorpamplona File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
31383
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No that's for the event, according to the old version of #784, 31382 is for private relationship between public keys
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but there is no reference to 31383 in this PR, so then it shouldn't be in the README either unless it is explained.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am confused, 31383 is not in the readme.
Public 30382
Private 31382