Last Call Review of draft-ietf-regext-org-11
review-ietf-regext-org-11-opsdir-lc-schoenwaelder-2018-10-19-00

Request Review of draft-ietf-regext-org
Requested rev. no specific revision (document currently at 12)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2018-10-08
Requested 2018-09-24
Other Reviews Secdir Last Call review of -10 by Charlie Kaufman (diff)
Genart Last Call review of -10 by Stewart Bryant (diff)
Genart Telechat review of -11 by Stewart Bryant (diff)
Review State Completed
Reviewer Jürgen Schönwälder
Review review-ietf-regext-org-11-opsdir-lc-schoenwaelder-2018-10-19
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/1o1e8waa1jbhrMObmeXZEM-8qrM
Reviewed rev. 11 (document currently at 12)
Review result Has Nits
Draft last updated 2018-10-19
Review completed: 2018-10-19

Review
review-ietf-regext-org-11-opsdir-lc-schoenwaelder-2018-10-19

Thanks for a concise and well written document. I have not run tools
to validate the XSD and the XML artefacts but the definitions all look
reasonable.

Comment:

- What does the following mean in the definition of e164StringType?

        <attribute name="x" type="token" />

  It is indeed the intention to use "x"? What is the reason for having
  e164Type, why not use e164StringType?

Nits:

- What are "Organization transform commands"? This phrase appears
  several times in 3.4 an it seems this refers to what section 4.2
  describes later on. Well, if you read the document sequentially,
  things are a bit confusing. Perhaps define upfront what transform
  commands are? It might also be that EPP people are used to
  "transform commands" and this just shows my lack of familiarity with
  EPP.

- s/ement/element/

- s/Zero of more <org:status> element/Zero of more <org:status> elements/