Re: [w3c/manifest] Add id member to manifest (#988)

@philloooo commented on this pull request.



> +          globally. When |id| is changed, it's recognized as a new app and
+          can be installed separately. When id is not changed, it can be used
+          to find installed application to perform manifest update.
+        </p>
+        <p>
+          The |id| can also be used by third party entities such as PWA stores
+          to uniquely identify and reference web apps
+        </p>
+        <p>
+          To <dfn>process the `id` member</dfn>, given [=object=]
+          |json:JSON|, [=ordered map=] |manifest:ordered map|.:
+        </p>
+        <ol class="algorithm">
+          <li>If the type of |json|["id"] is not [=string=], return.
+          </li>
+          <li>Let |id:URL| be the result of [=URL Parser|parsing=]

Yeah I was struggling with which approach to take in the beginning. But your suggestion does make sense. I will update the implementation too then

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/manifest/pull/988#discussion_r671442898

Received on Friday, 16 July 2021 18:15:31 UTC