New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add app.getApplicationInfoForProtocol API #24112
Conversation
💖 Thanks for opening this pull request! 💖 We use semantic commit messages to streamline the release process. Before your pull request can be merged, you should update your pull request title to start with a semantic prefix. Examples of commit messages with semantic prefixes:
Things that will help get your PR across the finish line:
We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me
@electron/wg-api reviewed and approved this at our June 29, 2020 meeting |
Congrats on merging your first pull request! 🎉🎉🎉 |
Release Notes Persisted
|
* pre merge * windows changes * added tests * clean up * more cleanup * lint error * windows 7 support * added windows 7 implementation * code review * lint and code review * code review * app.md merge conflict * merge conflict app.md accidently deleted code block * 'lint' * mis-moved getapplicationinfoforprotocol() into anonymous namespace * fix test * lint * code review
* pre merge * windows changes * added tests * clean up * more cleanup * lint error * windows 7 support * added windows 7 implementation * code review * lint and code review * code review * app.md merge conflict * merge conflict app.md accidently deleted code block * 'lint' * mis-moved getapplicationinfoforprotocol() into anonymous namespace * fix test * lint * code review
Description of Change
This api returns the name, display icon and path of the application registered as the default handler of a specific protocol (eg. the protocol 'https://example.com' would return 'Chrome', Chrome's icon and its installation path (or whichever your default browser happens to be).
It's an extension of the "getApplicationNameForProtocol" api to allow for more information on the application to be returned
Checklist
npm test
passesRelease Notes
Notes: Added
app.getApplicationInfoForProtocol()
API that returns detailed information about the app that handles a certain protocol