[Jmap] Mahesh Jethanandani's Discuss on draft-ietf-jmap-contacts-09: (with DISCUSS and COMMENT)
Mahesh Jethanandani via Datatracker <noreply@ietf.org> Tue, 21 May 2024 00:56 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: jmap@ietf.org
Delivered-To: jmap@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 48199C1C3D4F; Mon, 20 May 2024 17:56:57 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Mahesh Jethanandani via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
X-Test-IDTracker: no
X-IETF-IDTracker: 12.11.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <171625301728.10738.743002934797335476@ietfa.amsl.com>
Date: Mon, 20 May 2024 17:56:57 -0700
Message-ID-Hash: 6ZFPWIUWGURG75I6YGB33HTMGUPCBUQY
X-Message-ID-Hash: 6ZFPWIUWGURG75I6YGB33HTMGUPCBUQY
X-MailFrom: noreply@ietf.org
X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-jmap.ietf.org-0; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header
CC: draft-ietf-jmap-contacts@ietf.org, jmap-chairs@ietf.org, jmap@ietf.org, fenton@bluepopcorn.net
X-Mailman-Version: 3.3.9rc4
Reply-To: Mahesh Jethanandani <mjethanandani@gmail.com>
Subject: [Jmap] Mahesh Jethanandani's Discuss on draft-ietf-jmap-contacts-09: (with DISCUSS and COMMENT)
List-Id: JSON Message Access Protocol <jmap.ietf.org>
Archived-At: <https://mailarchive.ietf.org/arch/msg/jmap/-3I5DX5Tc8RYMojhXkGF8svUvKk>
List-Archive: <https://mailarchive.ietf.org/arch/browse/jmap>
List-Help: <mailto:jmap-request@ietf.org?subject=help>
List-Owner: <mailto:jmap-owner@ietf.org>
List-Post: <mailto:jmap@ietf.org>
List-Subscribe: <mailto:jmap-join@ietf.org>
List-Unsubscribe: <mailto:jmap-leave@ietf.org>
Mahesh Jethanandani has entered the following ballot position for draft-ietf-jmap-contacts-09: Discuss When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/about/groups/iesg/statements/handling-ballot-positions/ for more information about how to handle DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-jmap-contacts/ ---------------------------------------------------------------------- DISCUSS: ---------------------------------------------------------------------- Thanks for the document. It was short and easy to read. The following two DISCUSS should be easy to resolve with some additional text, or they could be my lack of understanding of how JMAP works, in which case I would not mind being educated. Section 2.3, paragraph 3 > If false, any attempt to destroy an AddressBook that still has > ContactCard in it will be rejected with an addressBookHasContents > SetError. If true, any ContactCards that were in the AddressBook > will be removed from it, and if in no other AddressBooks they will > be destroyed. The "if true" condition is not clear to me, especially if other AddressBooks exist. There is a distinction that is being drawn between removing a ContactCard and destroying it, which is not apparent. Can that distinction be explained? Related to that, it appears the ContactCards that were part of an AddressBook that is being deleted, will be moved if another AddressBook exists. Which AddressBook would that be? The default AddressBook? What happens if the default AddressBook is being removed? Section 3, paragraph 2 > * *addressBookIds*: Id[Boolean] > The set of AddressBook ids this ContactCard belongs to. A card > MUST belong to at least one AddressBook at all times (until it is > destroyed). The set is represented as an object, with each key > being an AddressBook id. The value for each key in the object > MUST be true. What happens if a ContactCard is removed from an AddressBook that is present in multiple AddressBooks? Is it removed from all AddressBooks, or just one? If it is removed from all AddressBooks, how does a user who wants to trim the AddressBook to certain contacts remove a ContactCard? ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- The IANA review of this document seems to not have concluded yet. ------------------------------------------------------------------------------- NIT ------------------------------------------------------------------------------- All comments below are about very minor potential issues that you may choose to address in some way - or ignore - as you see fit. Some were flagged by automated tools (via https://github.com/larseggert/ietf-reviewtool) so there will likely be some false positives. There is no need to let me know what you did with these suggestions. Section 2, paragraph 12 > w AddressBooks created by the user themself. If false, the AddressBook and it > ^^^^^^^^ Generally speaking, "themself" is only acceptable when referring to a singular entity (such as the singular usage of "they", which is the preferred pronoun for many non-binary people). If "themself" refers to a plural entity (such as "everybody", or the standard usage of "they"), you should use "themselves". Section 2.3, paragraph 3 > s, it MUST be ignored and the currently default AddressBook (if any) will rem > ^^^^^^^^^^^^^^^^^ You used an adverb ("currently") instead of an adjective, or a noun ("default") instead of another adjective.
- [Jmap] Mahesh Jethanandani's Discuss on draft-iet… Mahesh Jethanandani via Datatracker
- [Jmap] Re: Mahesh Jethanandani's Discuss on draft… Neil Jenkins
- [Jmap] Re: Mahesh Jethanandani's Discuss on draft… Mahesh Jethanandani
- [Jmap] Re: Mahesh Jethanandani's Discuss on draft… Neil Jenkins
- [Jmap] Re: Mahesh Jethanandani's Discuss on draft… Mahesh Jethanandani
- [Jmap] Re: Mahesh Jethanandani's Discuss on draft… Neil Jenkins