-
Notifications
You must be signed in to change notification settings - Fork 189
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
Restore contact names for did:erc725 (again) #373
Conversation
@peacekeeper @msporny asked for it to be reverted, I didn't see his feedback here: You replied:
|
@@ -2,8 +2,8 @@ | |||
"name": "erc725", | |||
"status": "registered", | |||
"verifiableDataRegistry": "Ethereum", | |||
"contactName": "Fabian Vogelsteller", | |||
"contactName": "Markus Sabadello, Fabian Vogelsteller, Peter Kolarov", |
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.
"contactName": "Markus Sabadello, Fabian Vogelsteller, Peter Kolarov", | |
"contactName": "Markus Sabadello", |
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.
contactName should be a single entity, either a person or an organization.
I think @msporny 's comment in #368 (comment) was about having a single contact email, not necessarily a single contact name. I agree with this. There are lots of DID methods that have multiple names as contact. This PR is only trying to fix a mistake that happened during the JSON conversion in #353, nothing else. If we want to remove multiple contact names, then this should be done in a separate PR, for all DID methods. |
@msporny please review, I reverted the merge based on you request, happy to merge if you agree with @peacekeeper . |
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 agree with Markus' corrections -- having three contact names is fine, the thing that's most important is the single contactEmail
, IMHO. I'm good w/ merging as-is.
ac53cd3
to
9e846df
Compare
03632eb
to
7f11dbe
Compare
Editorial, multiple reviews, no remaining objections, merging. |
Not sure why #368 was reverted in #371. Trying again.