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

add redactList for PaymentAddress (Part 1) #654

Merged
merged 35 commits into from
Mar 15, 2018
Merged
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
a10bb08
WIP: make PaymentAddress constructable
marcoscaceres Nov 20, 2017
5d1a35b
Address review feedback, contruct from dict
marcoscaceres Dec 6, 2017
8a4cbd8
fix typo
marcoscaceres Dec 6, 2017
124686f
Link strip and collapse ws
marcoscaceres Dec 6, 2017
b25a4d8
More WIP
marcoscaceres Dec 7, 2017
af7c9d7
typo
marcoscaceres Dec 7, 2017
db19219
Mandate phone numbers adhere to E.164 (closes #652)
marcoscaceres Dec 8, 2017
1101a7f
Editorial cleanup
marcoscaceres Dec 8, 2017
4b16b63
Relocated user-created address algo
marcoscaceres Dec 8, 2017
54a10eb
Link to enum value defintions
marcoscaceres Dec 8, 2017
92e04c7
Address privacy of post codes
marcoscaceres Dec 8, 2017
d7de7d4
Feat: add regionCode attribute and member
marcoscaceres Jan 24, 2018
76c8baa
review feedback
marcoscaceres Feb 13, 2018
d5d455f
make constructor internal
marcoscaceres Feb 27, 2018
4501c4d
tidy
marcoscaceres Feb 27, 2018
799a719
Make exclude list values optional
marcoscaceres Feb 27, 2018
66b9a3b
Not possible for devs to represent addresses
marcoscaceres Feb 27, 2018
10923f5
Remove AddressField enum
marcoscaceres Feb 27, 2018
24ff121
Remove phone number algos
marcoscaceres Feb 27, 2018
1f0d0e4
nit
marcoscaceres Feb 27, 2018
0378ab5
Relocate recipient format details
marcoscaceres Feb 27, 2018
67f4e7a
nits
marcoscaceres Feb 27, 2018
6f751d7
Editorial improvements
marcoscaceres Feb 27, 2018
f5ff02e
remove regionCode
marcoscaceres Feb 28, 2018
31428e7
s/excludeList/redactList
marcoscaceres Mar 6, 2018
1807bf4
Review nits/typos
marcoscaceres Mar 7, 2018
358f08b
fix construction details
marcoscaceres Mar 7, 2018
82eee2e
trim things
marcoscaceres Mar 7, 2018
d032e25
Add privacy note about redactList
marcoscaceres Mar 8, 2018
30e6806
assure request+response use same shippingAddress
marcoscaceres Mar 8, 2018
a7e615b
fix respec warning
marcoscaceres Mar 8, 2018
f968a0c
tidy
marcoscaceres Mar 8, 2018
50f6cdd
If not shipping, response address is null
marcoscaceres Mar 8, 2018
331280c
Review feedback
marcoscaceres Mar 14, 2018
e21719c
fix: typo
marcoscaceres Mar 15, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Address privacy of post codes
  • Loading branch information
marcoscaceres committed Mar 15, 2018
commit 92e04c7046a405f35ebbcdb0a774c67d5b968d81
42 changes: 27 additions & 15 deletions index.html
Original file line number Diff line number Diff line change
Expand Up @@ -2449,7 +2449,7 @@ <h2>
"PaymentAddress.PaymentAddress()">constructing</a> a
<a>PaymentAddress</a>. Its members are as follows.
</p>
<dl data-dfn-for="AddressInit" data-sort="ascending">
<dl data-dfn-for="AddressInit" data-link-for="" data-sort="ascending">
<dt>
<dfn>country</dfn> member
</dt>
Expand Down Expand Up @@ -2490,7 +2490,7 @@ <h2>
<dfn>sortingCode</dfn> member
</dt>
<dd>
A <a>sorting code</a>
A <a>sorting code</a>.
</dd>
<dt>
<dfn>languageCode</dfn> member
Expand Down Expand Up @@ -2579,7 +2579,8 @@ <h2>
</li>
<li>If "<a data-link-for="AddressField">phone</a>" is not in
<var>excludeList</var>, set <var>details</var>["<a>phone</a>"] to the
user-provided <a>structurally valid phone number</a>, or to the empty
user-provided <a>structurally valid phone number</a> in <a data-lt=
"canonicalize a phone number">canonical form</a>, or to the empty
string if none was provided.
<aside class="note" title="Privacy of phone number">
<p>
Expand Down Expand Up @@ -2614,7 +2615,19 @@ <h2>
<li>If "<a data-link-for="AddressField">postalCode</a>" is not in
<var>excludeList</var>, set <var>details</var>["<a>postalCode</a>"]
to the user-provided postal code, or to the empty string if none was
provided.
provided. Optionally, redact part of
<var>details</var>["<a>postalCode</a>"].
<div class="note" title="Privacy of Postal Codes">
<p>
<a>Postal codes</a> in certain countries can be so specific as to
uniquely identify an individual. This being a privacy concern,
some user agents only return the part of a postal code that they
deem sufficient for a merchant to calculate shipping costs. This
varies across countries and regions, and so the choice to redact
part, or all, of the postal code is left to the discretion of
implementers in the interest of protecting users' privacy.
</p>
</div>
</li>
<li>If "<a data-link-for="AddressField">recipient</a>" is not in
<var>excludeList</var>, set <var>details</var>["<a>recipient</a>"] to
Expand Down Expand Up @@ -3272,7 +3285,7 @@ <h2>
run the following steps:
<ol>
<li data-link-for="AddressField">Let <var>excludeList</var> be
the list « "<a>organization</a>", "<a>phoneNumber</a>",
the list « "<a>organization</a>", "<a>phone</a>",
"<a>recipient</a>", "<a>addressLine</a>" ».
</li>
<li>Let <var>address</var> be the result of running the steps to
Expand Down Expand Up @@ -3414,11 +3427,12 @@ <h2>
</li>
<li>If the <a data-lt=
"PaymentOptions.requestShipping">requestShipping</a> value of
<var>request</var>.<a>[[\options]]</a> is true, then set the
<a data-lt="PaymentResponse.shippingAddress">shippingAddress</a>
attribute of <var>response</var> to the value of the <a data-lt=
"PaymentRequest.shippingAddress">shippingAddress</a> attribute of
<var>request</var>. Otherwise, set it to null.
<var>request</var>.<a>[[\options]]</a> is true, then <a>create a
<code>PaymentAddress</code> from user-provided input</a> passing the
empty list as the <var>excludeList</var> and set the <a data-lt=
"PaymentResponse.shippingAddress">shippingAddress</a> attribute of
<var>response</var> to the resulting <a>PaymentAddress</a>.
Otherwise, set it to null.
</li>
<li>If the <a data-lt=
"PaymentOptions.requestShipping">requestShipping</a> value of
Expand Down Expand Up @@ -3446,11 +3460,9 @@ <h2>
"PaymentOptions.requestPayerPhone">requestPayerPhone</a> value of
<var>request</var>.<a>[[\options]]</a> is true, then set the
<a data-lt="PaymentResponse.payerPhone">payerPhone</a> attribute of
<var>response</var> to the payer's phone number provided by the user,
or to null if none was provided. When setting the <a data-lt=
"PaymentResponse.payerPhone">payerPhone</a> value, the user agent
SHOULD format the phone number to adhere to [[!E.164]]. Otherwise,
set it to null.
<var>response</var> to a <a>structurally valid phone number</a> in
<a data-lt="canonicalize a phone number">canonical form</a> provided
by the user, or to null if none was provided.
</li>
<li>Set <var>response</var>.<a>[[\completeCalled]]</a> to false.
</li>
Expand Down