Re: [Jmap] Working group last call: draft-ietf-jmap-mdn-03

["Neil Jenkins" <[e-mail]>]

Overall I think this doc is good. A few nits:

> o  forEmailId: "String" Email Id of the received email this MDN is
   relative to.

This needs to be nullable, as described in MDN/parse.

> o  originalMessageID: "String|null" (server-set)

I think this should be *originalMessageId* (lowercase d on the end) for consistency.

> *
> 2.1 <https://tools.ietf.org/html/draft-ietf-jmap-mdn-03#section-2.1>.  MDN/set

*
> 
   Standard "/set" method as described in [RFC8620 <https://tools.ietf.org/html/rfc8620>] where only the
   _create_ parameter is supported.

This is not quite right: the "accountId" and "ifInState" arguments are fine. I think this text should say something like:

> Only create is supported; any attempt to update/destroy MUST be rejected with a "forbidden" SetError.

And as a general comment, in the final versions of RFC8620/8621 we cleaned up the formatting so it worked better in the plain text version of the RFCs; I suggest adopting the same. So we use "quote" for a property name reference (not _underscore_) and remove the *asterisks* around the property names in the definitions.

Cheers,
Neil.