-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Updates old homepage_url key documentation #4291
Conversation
✅ Deploy Preview for developer-chrome-com ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Co-authored-by: Simeon Vincent <simeonv@google.com>
Co-authored-by: Simeon Vincent <simeonv@google.com>
Staging build for commit af04eac started at 2023-01-17T16:17:25Z completed after 6.40 minutes. https://pr-4291-static-dot-dcc-staging.uc.r.appspot.com/ The following pages likely changed with this PR: |
Co-authored-by: Joe Medley <jmedley@google.com>
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.
Yay for getting this updated! I left a couple of comments. From a technical POV, I think this is fine because I'm not aware of any real changes around MV3. I left a few more style-focused nits which is perhaps out of scope of a technical review but hopefully helpful.
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.
This all looks good, thanks for the update :)
@IanStanion-google, until we get this 'Stage' bug fixed, I'll go ahead and merge your stuff. |
Changes proposed in this pull request:
Updates old mv2 manifest key page for mv3