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

PWA rules #770

Open
poshaughnessy opened this issue Jan 22, 2018 · 3 comments
Open

PWA rules #770

poshaughnessy opened this issue Jan 22, 2018 · 3 comments

Comments

@poshaughnessy
Copy link
Contributor

I thought I'd create an issue to discuss PWA rules, overall?

A first question please: currently, the online scanner groups some rules (apple-touch-icon, manifest-file-extension, ...?) under a "PWA" section, but the CLI doesn't? Is the idea to have an explicit PWA category going forwards?

It would be good if we were able to cover our Samsung Internet 'ambient badging' rules, either with custom rules or hopefully if you agree, built-in ones?

Taking a quick look at each one:

How about other rules? (This is of course somewhat dependent on our PWA definition... Microsoft's slide here includes: Progressive, Responsive, Network Independent, App-Like, Fresh, Safe, Discoverable, Re-engageable, Installable, Linkable. And here is Google's checklist. I have some thoughts on this myself too).

Here are a couple more that I think might be suitable?:

  • Responsive design / mobile friendly
  • Service Worker caching (test that the site should function offline)
@alrra
Copy link
Contributor

alrra commented Jan 22, 2018

Is the idea to have an explicit PWA category going forwards?

We could have a formatter do that.

@sonarwhal/core ?

Also, related to this, we were thinking about having rule groups (see: #51). The idea being that users that want all the PWA rules for example, could simply include something in the lines of: "pwas": "error" and get all those rules.


It would be good if we were able to cover our Samsung Internet 'ambient badging' rules, either with custom rules or hopefully if you agree, built-in ones?

Personally, I think we should cover as much as possible related to PWAs with build-in/core rules.

"The site is served over HTTPS"
New rule required? I've seen the rules for 'strict-transport-security' headers and HTTP redirects, but not simply whether the page is served over HTTPS?

That should be a new rule.

How about other rules?
Here are a couple more that I think might be suitable?:

Can you open individual issues for each new rule you think we should have, and write a short description on what you think sonarwhal should test for, use cases, etc? Thanks!

@alrra
Copy link
Contributor

alrra commented Jan 22, 2018

Taking a quick look at each one:

In addition to those we are also planing on having a web app manifest parser integrated, which will make all the field related rules very simple to implement.

@alrra
Copy link
Contributor

alrra commented Mar 14, 2018

Is the idea to have an explicit PWA category going forwards?

Update: sonarwhal now supports the concept of configurations (e.g.: configuration-web-recommended), so that should take care of grouping things together.

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

No branches or pull requests

3 participants