Re: [w3c/push-api] Add Declarative Web Push (PR #385)

@saschanaz commented on this pull request.



> +                          |notification|, and |declarativeResult|'s [=declarative push message
+                          parser result/app badge=].
+                        </p>
+                      </li>
+                      <li>
+                        <p>
+                          If |result| is not failure, then set |notificationShown| to |result|'s
+                          [=push event result/notification shown=] and |appBadgeSet| to |result|'s
+                          [=push event result/app badge set=].
+                        </p>
+                      </li>
+                    </ol>
+                  </li>
+                  <li>
+                    <p>
+                      If |notificationShown| is false, then run the [=notification show steps=]

Noticed during an internal discussion; this would involve cookies as current implemented in Firefox, and clearing site data would mean the cookie jar would become empty, which may be significant if the image URL can only be fetched with proper cookie (imagine an image attachment from a DM). Should we accept such breakage? It seems like that should be somehow covered in the spec at least. (related: https://github.com/whatwg/notifications/issues/5)

-- 
Reply to this email directly or view it on GitHub:
https://github.com/w3c/push-api/pull/385#pullrequestreview-2624344558
You are receiving this because you are subscribed to this thread.

Message ID: <w3c/push-api/pull/385/review/2624344558@github.com>