{"action":"created","issue":{"url":"https://api.github.com/repos/w3ctag/design-reviews/issues/221","repository_url":"https://api.github.com/repos/w3ctag/design-reviews","labels_url":"https://api.github.com/repos/w3ctag/design-reviews/issues/221/labels{/name}","comments_url":"https://api.github.com/repos/w3ctag/design-reviews/issues/221/comments","events_url":"https://api.github.com/repos/w3ctag/design-reviews/issues/221/events","html_url":"https://github.com/w3ctag/design-reviews/issues/221","id":280028747,"node_id":"MDU6SXNzdWUyODAwMjg3NDc=","number":221,"title":"Web Share Target API","user":{"login":"mgiuca","id":228433,"node_id":"MDQ6VXNlcjIyODQzMw==","avatar_url":"https://avatars0.githubusercontent.com/u/228433?v=4","gravatar_id":"","url":"https://api.github.com/users/mgiuca","html_url":"https://github.com/mgiuca","followers_url":"https://api.github.com/users/mgiuca/followers","following_url":"https://api.github.com/users/mgiuca/following{/other_user}","gists_url":"https://api.github.com/users/mgiuca/gists{/gist_id}","starred_url":"https://api.github.com/users/mgiuca/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mgiuca/subscriptions","organizations_url":"https://api.github.com/users/mgiuca/orgs","repos_url":"https://api.github.com/users/mgiuca/repos","events_url":"https://api.github.com/users/mgiuca/events{/privacy}","received_events_url":"https://api.github.com/users/mgiuca/received_events","type":"User","site_admin":false},"labels":[{"id":350558879,"node_id":"MDU6TGFiZWwzNTA1NTg4Nzk=","url":"https://api.github.com/repos/w3ctag/design-reviews/labels/pending%20external%20feedback","name":"pending external feedback","color":"bfdadc","default":false}],"state":"open","locked":false,"assignee":{"login":"cynthia","id":43924,"node_id":"MDQ6VXNlcjQzOTI0","avatar_url":"https://avatars1.githubusercontent.com/u/43924?v=4","gravatar_id":"","url":"https://api.github.com/users/cynthia","html_url":"https://github.com/cynthia","followers_url":"https://api.github.com/users/cynthia/followers","following_url":"https://api.github.com/users/cynthia/following{/other_user}","gists_url":"https://api.github.com/users/cynthia/gists{/gist_id}","starred_url":"https://api.github.com/users/cynthia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cynthia/subscriptions","organizations_url":"https://api.github.com/users/cynthia/orgs","repos_url":"https://api.github.com/users/cynthia/repos","events_url":"https://api.github.com/users/cynthia/events{/privacy}","received_events_url":"https://api.github.com/users/cynthia/received_events","type":"User","site_admin":false},"assignees":[{"login":"cynthia","id":43924,"node_id":"MDQ6VXNlcjQzOTI0","avatar_url":"https://avatars1.githubusercontent.com/u/43924?v=4","gravatar_id":"","url":"https://api.github.com/users/cynthia","html_url":"https://github.com/cynthia","followers_url":"https://api.github.com/users/cynthia/followers","following_url":"https://api.github.com/users/cynthia/following{/other_user}","gists_url":"https://api.github.com/users/cynthia/gists{/gist_id}","starred_url":"https://api.github.com/users/cynthia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cynthia/subscriptions","organizations_url":"https://api.github.com/users/cynthia/orgs","repos_url":"https://api.github.com/users/cynthia/repos","events_url":"https://api.github.com/users/cynthia/events{/privacy}","received_events_url":"https://api.github.com/users/cynthia/received_events","type":"User","site_admin":false},{"login":"dbaron","id":248721,"node_id":"MDQ6VXNlcjI0ODcyMQ==","avatar_url":"https://avatars2.githubusercontent.com/u/248721?v=4","gravatar_id":"","url":"https://api.github.com/users/dbaron","html_url":"https://github.com/dbaron","followers_url":"https://api.github.com/users/dbaron/followers","following_url":"https://api.github.com/users/dbaron/following{/other_user}","gists_url":"https://api.github.com/users/dbaron/gists{/gist_id}","starred_url":"https://api.github.com/users/dbaron/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dbaron/subscriptions","organizations_url":"https://api.github.com/users/dbaron/orgs","repos_url":"https://api.github.com/users/dbaron/repos","events_url":"https://api.github.com/users/dbaron/events{/privacy}","received_events_url":"https://api.github.com/users/dbaron/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/w3ctag/design-reviews/milestones/165","html_url":"https://github.com/w3ctag/design-reviews/milestone/165","labels_url":"https://api.github.com/repos/w3ctag/design-reviews/milestones/165/labels","id":3222695,"node_id":"MDk6TWlsZXN0b25lMzIyMjY5NQ==","number":165,"title":"2018-06-12-telcon","description":"","creator":{"login":"plinss","id":2699689,"node_id":"MDQ6VXNlcjI2OTk2ODk=","avatar_url":"https://avatars3.githubusercontent.com/u/2699689?v=4","gravatar_id":"","url":"https://api.github.com/users/plinss","html_url":"https://github.com/plinss","followers_url":"https://api.github.com/users/plinss/followers","following_url":"https://api.github.com/users/plinss/following{/other_user}","gists_url":"https://api.github.com/users/plinss/gists{/gist_id}","starred_url":"https://api.github.com/users/plinss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/plinss/subscriptions","organizations_url":"https://api.github.com/users/plinss/orgs","repos_url":"https://api.github.com/users/plinss/repos","events_url":"https://api.github.com/users/plinss/events{/privacy}","received_events_url":"https://api.github.com/users/plinss/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":0,"state":"open","created_at":"2018-03-27T16:41:24Z","updated_at":"2018-05-22T15:25:04Z","due_on":"2018-06-12T07:00:00Z","closed_at":null},"comments":19,"created_at":"2017-12-07T07:28:52Z","updated_at":"2018-06-03T23:47:22Z","closed_at":null,"author_association":"NONE","body":"Hello TAG!\r\n\r\nI'm requesting a TAG review of:\r\n\r\n  - Name: Web Share Target\r\n  - Specification URL: https://wicg.github.io/web-share-target/\r\n  - Explainer, Requirements Doc, or Example code: https://github.com/WICG/web-share-target/blob/master/docs/explainer.md (please ignore examples; there are updated examples in the spec)\r\n  - Tests: None\r\n  - Primary contacts: @mgiuca, @owencm\r\n\r\nFurther details (optional):\r\n\r\n  - [x] I have reviewed the TAG's [API Design Principles](https://w3ctag.github.io/design-principles/)\r\n\r\nYou should also know that...\r\n\r\n- This is a companion to the [Web Share](https://wicg.github.io/web-share/) spec that was the subject of [this previous TAG review](https://github.com/w3ctag/design-reviews/issues/179). But the two specs are independent (other than sharing a common IDL data structure).\r\n- This is highly coupled to the [Web App Manifest](https://www.w3.org/TR/appmanifest/) spec, but I think it will remain as a separate spec, at least initially.\r\n- There is a prototype implementation in Chrome on Windows, Linux, Chrome OS and Android.\r\n- There is a demo site/app [here](https://wicg.github.io/web-share-target/demos/sharetarget.html).\r\n\r\nQuestions for the review panel:\r\n\r\n- The biggest open issue is [this](https://github.com/WICG/web-share-target/issues/25): we should catch errors in the URL template at manifest parse time. This is tricky because it essentially means we need to resolve the URL template against the manifest URL at parse time, before substituting the parameters. This isn't well defined because the URL template *is not a URL* and so can't have the URL parser applied to it. Not sure how to proceed; [my solution](https://github.com/WICG/web-share-target/pull/28) was to make a dummy parse by substituting the empty strings in, but it has issues noted in that pull request.\r\n- Related, we have [identified a problem](https://github.com/WICG/web-share-target/issues/30) if the template includes placeholders in path segments, like `\"/foo/{text}.\"`, if the text supplied is \"..\" then it would be a path escape. Not sure how to resolve this; escaping doesn't help because the URL Standard explicitly says that \"%2e%2e\" also means parent directory. The easiest solution is to simply prevent placeholders from appearing before the '?' (so they have to be in the query or fragment).\r\n\r\nWe'd prefer the TAG provide feedback as (please select one):\r\n\r\n  - [ ] open issues in our Github repo for each point of feedback\r\n  - [ ] open a single issue in our Github repo for the entire review\r\n  - [x] leave review feedback as a comment in this issue and @-notify @mgiuca\r\n\r\nThanks and feel free to post any follow-up discussion questions here."},"comment":{"url":"https://api.github.com/repos/w3ctag/design-reviews/issues/comments/394201696","html_url":"https://github.com/w3ctag/design-reviews/issues/221#issuecomment-394201696","issue_url":"https://api.github.com/repos/w3ctag/design-reviews/issues/221","id":394201696,"node_id":"MDEyOklzc3VlQ29tbWVudDM5NDIwMTY5Ng==","user":{"login":"mgiuca","id":228433,"node_id":"MDQ6VXNlcjIyODQzMw==","avatar_url":"https://avatars0.githubusercontent.com/u/228433?v=4","gravatar_id":"","url":"https://api.github.com/users/mgiuca","html_url":"https://github.com/mgiuca","followers_url":"https://api.github.com/users/mgiuca/followers","following_url":"https://api.github.com/users/mgiuca/following{/other_user}","gists_url":"https://api.github.com/users/mgiuca/gists{/gist_id}","starred_url":"https://api.github.com/users/mgiuca/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mgiuca/subscriptions","organizations_url":"https://api.github.com/users/mgiuca/orgs","repos_url":"https://api.github.com/users/mgiuca/repos","events_url":"https://api.github.com/users/mgiuca/events{/privacy}","received_events_url":"https://api.github.com/users/mgiuca/received_events","type":"User","site_admin":false},"created_at":"2018-06-03T23:47:22Z","updated_at":"2018-06-03T23:47:22Z","author_association":"NONE","body":"@sorenrehkopf As far as I know, Chrome Canary still supports Web Share Target. If it isn't working, please report to https://crbug.com/new and give relevant details. If you have questions about the project at an API level, please post them on https://github.com/WICG/web-share-target/issues. The current status of the design is as @ewilligers posted above 12 days ago."}}
{"actor":{"display_login":"mgiuca"}}