- changed status to open
[IdA Claims Registration] typos
Issue #1404
resolved
This is a review for “OpenID Connect for Identity Assurance Claims Registration 1.0”.
- Torsten’s affiliation in the “Authors” section should be changed from
yes.com
to his current organization. - “draft” should be removed from the specification title.
- “an id_token” → “an ID Token” (in the “Introduction” section)
- The “Warning” section should be removed from the final version.
- “claim provider” → “a claims provider” (twice in “3.2. identity proofing”)
- “claim provider” → “claims provider” (in “3.3. identity verification”)
- “claim provider” → “claims provider” (in “3.4. identity assurance”)
- “End-User data” → “end-user data” (in the first paragraph in “4.1. Additional claims about end-users”)
- “End-User’s place of birth” → “End-user’s place of birth” (in the description for
place_of_birth
in “4.1. Additional claims about end-users”) - The scheme of the link to the OIDC Core spec in the “Normative References” section should be changed from
http
tohttps
. - The scheme of the link to the OIDC4IDA spec in the “Normative References” section should be changed from
http
tohttps
. - “End-User’s salutation” → “End-user’s salutation” (in “A.1.1.6. Claim salutation”)
- The example, “Mr.”, in “A.1.1.6. Claim salutation” contains a period, but this is different from the example in “4.3. Examples”.
- “End-User’s title” → “End-user’s title” (in “A.1.1.7. Claim title”)
- The example, “Dr.”, in “A.1.1.7. Claim title” contains a period, but this is different from the example in “4.3. Examples”.
- “End-User’s mobile” → “End-user’s mobile” (in “A.1.1.8. Claim msisdn”)
- Torsten’s affiliation in the “Author’s Addresses” should be changed from
yes.com
to his current organization.
Comments (6)
-
-
- changed component to Assurance
- changed milestone to IDA Final
-
assigned issue to
-
Thanks Taka!
Here are my opinions of these items
- Torsten’s affiliation in the “Authors” section should be changed from
yes.com
to his current organization. - - “draft” should be removed from the specification title. - not until it is approved by foundation wide vote
- “an id_token” → “an ID Token” (in the “Introduction” section) -
- The “Warning” section should be removed from the final version. - agreed but not until it is approved by foundation wide vote
- “claim provider” → “a claims provider” (twice in “3.2. identity proofing”) I think we settled on singular because of ISO guidance and translation challenges of plurals
- “claim provider” → “claims provider” (in “3.3. identity verification”) - I think we settled on singular because of ISO guidance and translation challenges of plurals
- “claim provider” → “claims provider” (in “3.4. identity assurance”) - I think we settled on singular because of ISO guidance and translation challenges of plurals
- “End-User data” → “end-user data” (in the first paragraph in “4.1. Additional claims about end-users”) -
- “End-User’s place of birth” → “End-user’s place of birth” (in the description for
place_of_birth
in “4.1. Additional claims about end-users”) - - The scheme of the link to the OIDC Core spec in the “Normative References” section should be changed from
http
tohttps
. - - The scheme of the link to the OIDC4IDA spec in the “Normative References” section should be changed from
http
tohttps
. - - “End-User’s salutation” → “End-user’s salutation” (in “A.1.1.6. Claim salutation”) -
- The example, “Mr.”, in “A.1.1.6. Claim salutation” contains a period, but this is different from the example in “4.3. Examples”. - need to decide which is correct (probably without)
- “End-User’s title” → “End-user’s title” (in “A.1.1.7. Claim title”) -
- The example, “Dr.”, in “A.1.1.7. Claim title” contains a period, but this is different from the example in “4.3. Examples”.-
- “End-User’s mobile” → “End-user’s mobile” (in “A.1.1.8. Claim msisdn”) -
- Torsten’s affiliation in the “Author’s Addresses” should be changed from
yes.com
to his current organization. -
- Torsten’s affiliation in the “Authors” section should be changed from
-
reporter If “a claim provider” should not be changed to “a claims provider” for ISO, then, all occurrences of “a claims provider” in “OpenID for Identity Assurance schema definition 1.0” need to be replaced with “a claim provider”.
The reason I suggested replacing “a claim provider with “a claims provider” was that I started my reviewing from “OpenID for Identity Assurance schema definition 1.0”.
-
Issue
#1406raised to address plural “claims provider” in schema doc -
- changed status to resolved
PR#206 merged
- Log in to comment