[Gen-art] Genart last call review of draft-ietf-calext-eventpub-extensions-12

Dan Romascanu via Datatracker <noreply@ietf.org> Sun, 28 April 2019 09:58 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: gen-art@ietf.org
Delivered-To: gen-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id BB6EF12004F; Sun, 28 Apr 2019 02:58:38 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Dan Romascanu via Datatracker <noreply@ietf.org>
To: <gen-art@ietf.org>
Cc: draft-ietf-calext-eventpub-extensions.all@ietf.org, ietf@ietf.org, calsify@ietf.org, dromasca@gmail.com
X-Test-IDTracker: no
X-IETF-IDTracker: 6.95.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Dan Romascanu <dromasca@gmail.com>
Message-ID: <155644551863.20986.430303451434247977@ietfa.amsl.com>
Date: Sun, 28 Apr 2019 02:58:38 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/ie0ySHcSX4D3O4HmsPVjYJzcQk8>
Subject: [Gen-art] Genart last call review of draft-ietf-calext-eventpub-extensions-12
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 28 Apr 2019 09:58:39 -0000

Reviewer: Dan Romascanu
Review result: Ready with Issues

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-calext-eventpub-extensions-12
Reviewer: Dan Romascanu
Review Date: 2019-04-28
IETF LC End Date: 2019-04-29
IESG Telechat date: Not scheduled for a telechat

Summary:

This is a clear and detailed document updating RFC 5545 to include new
properties and components to iCalendar. The document is READY for publication.
I found a number of minor issues as well as a few nits which should be
clarified and fixed in the further editing process.

Major issues:

Minor issues:

1. The document uses the term 'event' which has many meanings in many contexts.
It would be useful to include or reference a definition of what is meant by
'event' in the context of iCalendar

2. The Abstract mentions that some of the updates are useful for social
networking. I could not find any support for this claim in the text, with the
exception mybe of mentioning 'social calendaring' in Section 3, which is also a
vague term (or at least I do not know where it is defined or referred in the
IETF). I suggest to either clarify, or drop these mentions.

3. In Section 3:

> Using STRUCTURED-LOCATION, information about a number of interesting
   locations can be communicated, for example, address, region, country,

Isn't it rather 'supplementary information about the location' than
'information about a number of interesting locations'?

4. Inconsistent use/non-use of keywords.

In 5.2:

'This parameter MAY be specified on STRUCTURED-RESOURCE
      and provides a way to differentiate multiple properties.'

while in 5.5:

'This property parameter can be specified on any
      property when the value is derived from some other property or
      properties.'

I suggest to decide on a consistent use of either MAY or 'can'.

Nits/editorial comments:

1. In Section 2:

> This is a
   better match for the way [W3C.REC-xml-20081126] and JSON, [RFC8259]
   handles such structures and allows richer definitions.

Fix grammar (plural)

2. In the use cases in Section 3, there are many optional parameters, and the
use of conditional in the text may be more appropriate.

For example in 3.1.1:

'In addition, there will be sponsorship information for
   sponsors of the event'

better

'In addition, there may be sponsorship information for
   sponsors of the event'

3. In Section 3.1.2.1

s/A meeting may have 10 attendees non of which are co-located/A meeting may
have 10 attendees, none of which are co-located/

4. I do not know if Appendix B will be kept, but in case it will be, there is a
typo in:

> Fix PARTTYPE/PARTICPANT-TYPE inconsistency