Skip to content

Conversation

@adrianaixba
Copy link
Collaborator

@adrianaixba adrianaixba commented Sep 12, 2023

updated report with PWA category
updated report with no PWA category

checks off the following in issue/5535:

  • remove/rearrange PWA audits
  • remove the category from the json result
  • ensure support for report PWA category rendering post removal
  • Get rid of Manifest/InstallabilityErrors gatherers and ManifestValues computed artifact

@adamraine adamraine added this to the v12.0 milestone Sep 12, 2023
{id: 'valid-source-maps', weight: 0, group: 'best-practices-general'},
{id: 'inspector-issues', weight: 1, group: 'best-practices-general'},
{id: 'themed-omnibox', weight: 1, group: 'best-practices-general'},
{id: 'viewport', weight: 2, group: 'best-practices-general'},
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we need this audit in performance, seo and best practices. I think we should remove it from SEO if we really want it here, but that's a different conversation.

@connorjclark
Copy link
Collaborator

found more PWA references:

  • pwaCategoryTitle and other pwa strings in default-config
  • PWA in some stale comments in esm-config / a11y.js
  • "How is the PWA (Progressive Web App) score calculated?" in scoring.md
  • "categoryProgressiveWebApp' ui-strings.js
  • a comment in description lighthouse-result.proto
  • report-utils.js
  • InstallabilityErrors type in artifacts.d.ts

@connorjclark
Copy link
Collaborator

  • docs/architecture.md mentions PWA
  • docs/understanding-results.md

/** The issues surfaced in the devtools Issues panel */
InspectorIssues: Artifacts.InspectorIssues;
/** Errors preventing page being installable as PWA. */
InstallabilityErrors: Artifacts.InstallabilityErrors;
Copy link
Contributor

Choose a reason for hiding this comment

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

WebAppManifest too?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done!

@adrianaixba adrianaixba merged commit 7187606 into main Apr 10, 2024
@adrianaixba adrianaixba deleted the rm-pwa branch April 10, 2024 23:08
@adamraine adamraine mentioned this pull request Jul 8, 2024
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.

4 participants