Hi Joel,

Thanks for the review.


On Sat, 16 May 2020 at 18:35, Joel Halpern via Datatracker
<nore...@ietf.org> wrote:
>

>
> Summary: This document is almost ready for publication as an Informational RFC
>
> Major issues:
>     This document says it is Informational.  It says it describes "a" capport
>     architecture.  But the third paragraph of the introduction says that this
>     document standardizes an architecture.  The rest of this review assumes
>     that is an error, and this is describing "an" architecture, rather than
>     "the IETF" architecture.
>

You are correct. This is an error. We will address it.


> Minor issues:
>     The abstract really should expand "capport".   As simple as having the
>     first sentence read "This document describes a "captive portal" (capport)
>     archtiecture."
>

Makes sense. Thanks

> Nits/editorial comments:
>     Following the first bulleted list in the introduction, the document reads:
>     "this document does not yet describe...?  The word "yet" seems
>     inappropriate.  We are pbulsihgin this as an RFC.  Please remove the 
> "yet".
>
>
>

Agreed. We'll remove it.

Thanks again for the feedback. I've made sure each item is captured in
a github issue so we don't forget to address them.

_______________________________________________
Captive-portals mailing list
Captive-portals@ietf.org
https://www.ietf.org/mailman/listinfo/captive-portals

Reply via email to