That looks good to me, Michael.
Barry
On Sun, Sep 20, 2020 at 10:35 PM Michael Slusarz <michael.slusarz@xxxxxxxxxxxxxxxx> wrote:
How about changing the first paragraph of Section 3.2 to the following:
"The server returns a variable-length string that is the generated preview for that message. This string is intended to be viewed by the user as a contextual preview of the entire message, and is not intended to be interpreted in any way by the client software."
Slightly altered Barry's wording - "user" is already defined in Section 2 as the human user, so to be consistent I think we should use "user" instead of "end-user".
michael
Meral, thanks for the review. The document does say in the first sentence of the Introduction that the preview is for the end user. I agree that it would be useful to emphasize that, though, and I think the right place to do it is by adding to the first paragraph of Section 3.2 something like the following: “The preview string is meant to help the end-user, and is not intended to be interpreted in any way by the client software.”
Barry
Reviewer: Meral Shirazipour
Review result: Ready with Nits
I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair. Please treat these comments just
like any other last call comments.
For more information, please see the FAQ at
<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
Document: draft-ietf-extra-imap-fetch-preview-09
Reviewer: Meral Shirazipour
Review Date: 2020-09-15
IETF LC End Date: 2020-09-14
IESG Telechat date: Not scheduled for a telechat
Summary: This draft is ready to be published as Standards Track RFC but I have
some comments.
Major issues:
Minor issues:
Nits/editorial comments: In Section 4.2, it would have been good to give some
example and advice on when the client uses the preview to take some action. Is
this to be avoided? (Was not very clear to me).
Best Regards,
Meral Shirazipour
www.ericsson.com
-- last-call mailing list last-call@xxxxxxxx https://www.ietf.org/mailman/listinfo/last-call
