decentralized-identity / didcomm-messaging

https://identity.foundation/didcomm-messaging/spec/
Apache License 2.0
161 stars 56 forks source link

remove conflict with issue #351 #373

Closed dhh1128 closed 2 years ago

dhh1128 commented 2 years ago

Signed-off-by: dhh1128 daniel.hardman@gmail.com

TelegramSam commented 2 years ago

This is a small but important change. Removing this allows for future work without conflict. I vote merge.

dhh1128 commented 2 years ago

I added one more change. I think with this additional change, this PR fixes #351 (although it doesn't fully resolve @TimoGlastra 's issue; it just lets it be fixed somewhere other than this spec).

dhh1128 commented 2 years ago

fixed #351

mccown commented 2 years ago

I like these changes. Given the consensus, I'll go ahead and merge.