Add manifest.launch_handler into web-platform-dx features #2701
Conversation
github-actions
bot
added
the
feature definition
|
Hi! Please take a look at this CL, this is my first one and I need approval from a maintainer to start the workflow. I'm a Software Engineer from Chrome looking to add some features here around the PWA area. I based this off feedback from #2395. |
|
@ddbeck and @captainbrosset seems like I do not have the necessary privileges for assigning someone to review my CL or trigger the workflow. Can I get your help again please? Thanks! |
features/app-launch-handler.yml
Outdated
| @@ -0,0 +1,9 @@ | |||
| name: Launch Handler | |||
There was a problem hiding this comment.
Hmm, we pluralize other "handlers" for web app manifests. @captainbrosset maybe you have an opinion about this?
Also, a nit: we use title case for feature names
| name: Launch Handler | |
| name: Launch handlers |
(If this change is accepted, then the ID for the feature should change to match.)
There was a problem hiding this comment.
Will wait on @captainbrosset 's feedback here.
This CL adds a web-platform-dx entry for launch_handler in manifest.
Spec: https://wicg.github.io/web-app-launch/