Skip to content
This repository was archived by the owner on Mar 14, 2024. It is now read-only.

Update index.md - #2773

Closed
Baggerone wants to merge 2 commits into
GoogleChrome:mainfrom
Baggerone:patch-1
Closed

Baggerone wants to merge 2 commits into
GoogleChrome:mainfrom
Baggerone:patch-1

Conversation

@Baggerone

Copy link
Copy Markdown

demo web page is no longer available

demo web page is no longer available
@google-cla

google-cla Bot commented May 6, 2022

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

For more information, open the CLA check for this pull request.

@netlify

netlify Bot commented May 6, 2022 •

Copy link
Copy Markdown

✅ Deploy Preview for developer-chrome-com ready!

Name Link
🔨 Latest commit 0755c61
🔍 Latest deploy log https://app.netlify.com/sites/developer-chrome-com/deploys/627ac330d8db89000890c435
😎 Deploy Preview https://deploy-preview-2773--developer-chrome-com.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@chrome-devrel-review-bot

chrome-devrel-review-bot commented May 6, 2022 •

Copy link
Copy Markdown
Collaborator

Hello! This is an automated review by our custom reviewbot. It updates automatically when code or GitHub comments in this pull request are created or updated.

Requested changes

If there are any common problems with the content files you created or modified, they will be listed here.

site/en/docs/devtools/webauthn/index.md

  • Please change your usage of the following words:
    • Avoid please. Affected lines: 22

@jpmedley jpmedley added P2 A normal priority task. This is the default for most issues. content update labels May 6, 2022
@alexandrascript

Copy link
Copy Markdown
Contributor

Thank you @Baggerone for the fix! Can you please sign the CLA, so we can get the updates merged?

For more information, open the CLA check for this pull request.

@alexandrascript

Copy link
Copy Markdown
Contributor

@sofiayem I assumed the WebAuthn was my coverage, but the docs are in devtools. Doesn't matter either way, just wanted to tell you I saw it and I'll take care of it unless there's a reason not to!

@Baggerone

Copy link
Copy Markdown
Author

This got fixed via a response to an issue I created

@Baggerone Baggerone closed this May 10, 2022
@Baggerone
Baggerone deleted the patch-1 branch May 10, 2022 20:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

content update P2 A normal priority task. This is the default for most issues.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants