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

Add LightWallet article #999

Merged
merged 3 commits into from
Jun 14, 2019
Merged

Add LightWallet article #999

merged 3 commits into from
Jun 14, 2019

Conversation

khempenius
Copy link
Contributor

No description provided.

@googlebot googlebot added the cla: yes Contributor has signed the CLA label Jun 11, 2019
Copy link
Contributor

@Meggin Meggin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After you make the fixes, make sure to run npm run dev and test the article in the browser.

src/site/content/en/fast/fast.11tydata.js Show resolved Hide resolved
@@ -0,0 +1,65 @@
# Use Lighthouse for Performance Budgets

Lighthouse now supports performance budgets. This feature, LightWallet, can be set up in under five minutes and provides feedback on the size and quantity of page resources.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a link to LightWallet somewhere?

@Meggin
Copy link
Contributor

Meggin commented Jun 13, 2019

Also, Rob pointed out that when you submit, you get a staged version. You can see it the checks.

@robdodson
Copy link
Contributor

Post should also have a 1600 x 800 px hero image.

You can copy YAML from this template post: https://github.com/GoogleChrome/web.dev/blob/master/src/site/_drafts/_template-post/index.md

@netlify
Copy link

netlify bot commented Jun 14, 2019

Deploy preview for web-dev-staging ready!

Built with commit 9141a22

https://deploy-preview-999--web-dev-staging.netlify.com

@khempenius
Copy link
Contributor Author

Updated w/ changes. I talked with @robdodson and I will not be adding a hero image to this article.

@robdodson robdodson merged commit f7964dd into master Jun 14, 2019
@robdodson robdodson deleted the lightwallet branch June 14, 2019 21:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes Contributor has signed the CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants