Skip to content

feat: add surveys / polls#111

Closed
fernandolguevara wants to merge 4 commits intonostr-protocol:masterfrom
fernandolguevara:fr-surveys
Closed

feat: add surveys / polls#111
fernandolguevara wants to merge 4 commits intonostr-protocol:masterfrom
fernandolguevara:fr-surveys

Conversation

@fernandolguevara
Copy link
Copy Markdown
Contributor

@fernandolguevara fernandolguevara commented Dec 22, 2022

#65

@fiatjaf what number could we use for this nip ?

@fiatjaf
Copy link
Copy Markdown
Member

fiatjaf commented Dec 22, 2022

41 I guess.

But this NIP is confusing to me. Why do relays have to know about the status of a poll? I guess this could use the parametrized replaceable events thing.

Also why are you defining t tags, p tags and other stuff on the poll answer event? These could be used, but they shouldn't be specified by this NIP, unless I'm missing something.

@fernandolguevara
Copy link
Copy Markdown
Contributor Author

@fiatjaf t and p tags are on the json examples just to show how this feature extend kind 1 events.

I will check if it is possible to use parametrized replaceable events

@Giszmo Giszmo mentioned this pull request Jan 17, 2023
@0xtlt
Copy link
Copy Markdown
Contributor

0xtlt commented Jan 17, 2023

I just saw this NIP proposal, I made another #148, I think we can try to merge our 2 ideas :)

So...

@fernandolguevara fernandolguevara requested a review from 0xtlt March 3, 2023 22:32
@vitorpamplona
Copy link
Copy Markdown
Collaborator

Replaced by NIP-88

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.

5 participants