Skip to content
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

Add lighthouse metrics #23

Open
martinschierle opened this issue Apr 23, 2018 · 2 comments
Open

Add lighthouse metrics #23

martinschierle opened this issue Apr 23, 2018 · 2 comments

Comments

@martinschierle
Copy link

Hey!

This is an amazing tool, thanks for creating this. One feature request though - could you add in the new lighthouse metrics? It should be somewhat easy - add 'lighthouse=1' to the API request (see here), and then add the metrics to the resultmap:

N Lighthouse Performance Score data.median.firstView['lighthouse.Performance']
O Lighthouse PWA Score data.median.firstView['lighthouse.ProgressiveWebApp']
P Lighthouse SEO Score data.median.firstView['lighthouse.SEO']
Q Lighthouse Accessibility data.median.firstView['lighthouse.Accessibility']
R Lighthosue Best Practices Score data.median.firstView['lighthouse.BestPractices']
S Lighthouse Meaningful Paint data.median.firstView['lighthouse.Performance.first-meaningful-paint']
T Lighthouse First Interactive data.median.firstView['lighthouse.Performance.first-interactive']
U Lighthouse Consist. Interactive data.median.firstView['lighthouse.Performance.consistently-interactive']

Regards,
Martin

@andydavies
Copy link
Contributor

(Sorry missed this issue before)

Probably makes sense to include an example of the how to get the lighthouse metrics in the default sheet as they are different to the others i.e. there's not a lighthouse object but a series of properties on first view.

Doing some updates to the sheet at the moment and will include an example

@andydavies
Copy link
Contributor

I've updated the template to include an example of how to get the data from the lighthouse audit.

Will update the readme to include this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants