-
Notifications
You must be signed in to change notification settings - Fork 4
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
Adds addressRegion to the address claim table for international addresses #305
Conversation
@@ -296,6 +296,7 @@ Each JSON object in the list may contain any of the following properties: | |||
| `addressLocality` | Maps to `POST_TOWN` in the [Postcode Address File][external.postcode-file] and [Ordnance Survey Places API][external.os-places-api]. | | |||
| `postalCode` | Maps to `POST_CODE` in the [Postcode Address File][external.postcode-file] and [Ordnance Survey Places API][external.os-places-api]. | | |||
| `addressCountry` | Two-letter [ISO 3166-1 alpha-2 country code](https://en.wikipedia.org/wiki/ISO_3166-1_alpha-2). | | |||
| `addressRegion` | The region in which the locality is, and which is in the country, provided as text. For example, California, or another appropriate first-level Administrative division. | |
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.
Could we include something to say that it aligns to https://schema.org/addressRegion? eg.
| `addressRegion` | The region in which the locality is, and which is in the country, provided as text. For example, California, or another appropriate first-level Administrative division. | | |
| `addressRegion` | The region in which the locality is, and which is in the country, provided as text. For example, California, or another appropriate first-level Administrative division. This maps to [schema:addressRegion](https://schema.org/addressRegion) | |
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.
Done
ab2ae80
to
55645e7
Compare
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.
Looks good to me and the inclusion of the maps to keeps it consistent with the other attributes.
source/integrate-with-integration-environment/prove-users-identity.html.md.erb
Outdated
Show resolved
Hide resolved
Co-authored-by: SteveEvans-GDS <[email protected]>
source/integrate-with-integration-environment/prove-users-identity.html.md.erb
Show resolved
Hide resolved
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.
Just the one query about Administrative division but otherwise looks really good!
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.
LGTM
Do not merge this PR until we've got approval from the policy board that we can go ahead
Why
To ensure the docs about VCs reflect changes for international addresses
What
Adding
addressRegion
to address claim tableTechnical writer support
Yes
How to review
Changes can be viewed locally at http://localhost:4567/integrate-with-integration-environment/prove-users-identity/#understand-your-user-s-address-claim
Confirm