Skip to content

report: update link for budgets audit#10944

Merged
devtools-bot merged 2 commits into
masterfrom
budgets-link
Jun 16, 2020
Merged

report: update link for budgets audit#10944
devtools-bot merged 2 commits into
masterfrom
budgets-link

Conversation

@kaycebasques

Copy link
Copy Markdown
Contributor

Summary

Updates the "Learn more" link in the budgets audit to avoid a redirect.

@kaycebasques kaycebasques requested a review from a team as a code owner June 10, 2020 22:44
@kaycebasques kaycebasques requested review from paulirish and removed request for a team June 10, 2020 22:44
@kaycebasques kaycebasques marked this pull request as draft June 10, 2020 22:46
@kaycebasques

kaycebasques commented Jun 10, 2020

Copy link
Copy Markdown
Contributor Author

On second look I'm realizing that since this is exposed by default in the DevTools report UI (I think??) I think we need a dedicated guide for this topic. It's currently linking to Katie's doc, which assumes that someone is configuring the CLI to use budgets.

@connorjclark

Copy link
Copy Markdown
Collaborator

On second look I'm realizing that since this is exposed by default in the DevTools report UI (I think??) I think we need a dedicated guide for this topic.

What do you mean by exposed by default?

You can't set a budget in DevTools at the moment.

image

so it's just gonna be this diagnostic with a link to budgets, a feature only supported in CLI atm.

@connorjclark connorjclark left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

+1 but need to run yarn update:sample-json

@paulirish paulirish changed the title Update "Learn more" link for budgets audit report: update link for budgets audit Jun 16, 2020
@paulirish paulirish marked this pull request as ready for review June 16, 2020 23:07
@devtools-bot devtools-bot merged commit e7e351c into master Jun 16, 2020
@devtools-bot devtools-bot deleted the budgets-link branch June 16, 2020 23:10
makunde pushed a commit to makunde/lighthouse that referenced this pull request Jul 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants