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

@marcoscaceres commented on this pull request.



> +              <td>"https://example.com/resources/start"</td>
+            </tr>
+            <tr>
+              <td>""</td>
+              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/"</td>
+            </tr>
+            <tr>
+              <td>"/"</td>
+              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/"</td>
+            </tr>
+            <tr>
+              <td>"foo"</td>
+              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/foo"</td>

This one is also incorrect, I believe: 

```suggestion
              <td>"https://example.com/resources/start/foo"</td>
```

-- 
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#pullrequestreview-759317231

Received on Tuesday, 21 September 2021 05:36:35 UTC