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.

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

Proposal for Survey protocol #109

Merged
merged 10 commits into from
Aug 12, 2024

Conversation

stephanbruijnis
Copy link
Contributor

A protocol where a survey (JSON Forms) is sent by the requester to the responder. The responder then replies with the answers or declines the survey.

stephanbruijnis and others added 8 commits May 16, 2024 09:56
Initial version readme.md version 0.1 for proposed survey protocol. Still several sections to complete
Read me updated with messages for survey 0.1 protocol
Rewrote the tutorial section of the read me, added example messages
Change the response message to contain a response_type indicating if the survey response will contain data (answers) or a decline
Add survey_id to the request message
Update reference implementation information
Corrected link label
Copy link
Member

@TheTechmage TheTechmage left a comment

Choose a reason for hiding this comment

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

Discussed in DIDComm UG 20240610

Copy link
Collaborator

@TelegramSam TelegramSam left a comment

Choose a reason for hiding this comment

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

This is great.

@TheTechmage
Copy link
Member

Looks like the primary issue is that the title attribute is Survey Protocol instead of just Survey

Change title from Survey Protocol to Survey
Add Jorge Flores as author
@stephanbruijnis
Copy link
Contributor Author

I've updated the title and authors attributes

Copy link
Contributor

@TimoGlastra TimoGlastra left a comment

Choose a reason for hiding this comment

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

Nice, i like the addition of this protocol! And I also like the re-use of JSON Schema with JSON Schema Forms, makes the protocol a lot easier.

Maybe it is worth it to add a mention of the Question Answer protocol and how this is an advanced version of that protocol?

site/content/protocols/survey/0.1/readme.md Outdated Show resolved Hide resolved
site/content/protocols/survey/0.1/readme.md Show resolved Hide resolved
site/content/protocols/survey/0.1/readme.md Outdated Show resolved Hide resolved
site/content/protocols/survey/0.1/readme.md Outdated Show resolved Hide resolved
@stephanbruijnis
Copy link
Contributor Author

Thank you @TimoGlastra for your feedback, it should be JSON objects instead of strings, I will update the protocol

@jorgefl0
Copy link
Contributor

Is there anything holding up approval of this PR?

Copy link
Collaborator

@TelegramSam TelegramSam left a comment

Choose a reason for hiding this comment

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

DIDComm WG discussion 20240812

@TelegramSam TelegramSam merged commit e02d305 into decentralized-identity:main Aug 12, 2024
1 check passed
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