-
Notifications
You must be signed in to change notification settings - Fork 541
Add MDN links, update web.dev & developer.chrome.com links #3552
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
base: main
Are you sure you want to change the base?
Conversation
This reverts commit ef4620c.
…ser/mikehoffms/mdn-links
microsoft-edge/extensions/developer-guide/migrate-your-extension-from-manifest-v2-to-v3.md
Outdated
Show resolved
Hide resolved
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
microsoft-edge/progressive-web-apps/how-to/window-controls-overlay.md
Outdated
Show resolved
Hide resolved
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
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.
LGTM - Content Developer
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
Learn Build status updates of commit d5a1c2b: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. microsoft-edge/devtools/whats-new/2021/04/devtools.md
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
## Background service workers | ||
|
||
Service workers are available for testing in the Canary preview channel of Microsoft Edge. To migrate your extensions from background pages to service workers, see [Migrate to a service worker](https://developer.chrome.com/docs/extensions/develop/migrate/to-service-workers). The Microsoft Edge extensions team is evaluating and investigating the impact that this change brings to both developers and users. | ||
To migrate your extensions from background pages to service workers, see [Migrate to a service worker](https://developer.chrome.com/docs/extensions/develop/migrate/to-service-workers)<!-- chrome link ok -->. |
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.
@G-u-r-j-e-e-t please verify that this change is ok:
- Removed sentence specifying Edge Canary.
- Removed sentence about Extensions team evaluating.
Rendered article section:
- Migrate an extension from Manifest V2 to V3 > Background service workers
- Internal preview: https://review.learn.microsoft.com/microsoft-edge/extensions/developer-guide/migrate-your-extension-from-manifest-v2-to-v3?branch=pr-en-us-3552#background-service-workers
- External preview: https://github.com/MicrosoftDocs/edge-developer/blob/user/mikehoffms/mdn-links/microsoft-edge/extensions/developer-guide/migrate-your-extension-from-manifest-v2-to-v3.md#background-service-workers
- Before/live: https://learn.microsoft.com/microsoft-edge/extensions/developer-guide/migrate-your-extension-from-manifest-v2-to-v3#background-service-workers
Rendered articles for review:
Performance tool: Analyze your website's performance
/devtools/performance/overview.md
& other filesMigrate an extension from Manifest V2 to V3 > Background service workers
/extensions/developer-guide/migrate-your-extension-from-manifest-v2-to-v3.md
AB#57905679