-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Conversation
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 changesIf there are any common problems with the content files you created or modified, they will be listed here.
|
✔️ Deploy Preview for web-dev-staging ready! 🔨 Explore the source changes: 8a7b20c 🔍 Inspect the deploy log: https://app.netlify.com/sites/web-dev-staging/deploys/61dc06ba3704fe0008310403 😎 Browse the preview: https://deploy-preview-7006--web-dev-staging.netlify.app |
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.
Thomas,
I had more content questions than usual. Since this is such a new topic, I think they're merited. Please consider them carefully.
Joe
Co-authored-by: Joe Medley <jmedley@google.com> Co-authored-by: Jeremy Wagner <malchata@users.noreply.github.com>
Co-authored-by: Joe Medley <jmedley@google.com>
@jpmedley Thanks for the thorough review. I have addressed all your remarks! PTAL. Thanks! |
Fixes #6947.
Changes proposed in this pull request: