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

feat: Added noscript notice to index.template.html #68

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

yusufkandemir
Copy link
Member

What kind of change does this PR introduce?

  • Feature

Does this PR introduce a breaking change?

  • No

The PR fulfills these requirements:

  • It's submitted to the dev branch and not the master branch

Other information:
Inspired from Vue CLI.

@sebastien-prudhomme
Copy link

Lighthouse checks that there is such a message in PWA so 👍

Copy link
Contributor

@Blfrg Blfrg left a comment

Choose a reason for hiding this comment

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

Please consider the change recommended here.

template/src/index.template.html Outdated Show resolved Hide resolved
@yusufkandemir yusufkandemir force-pushed the feat-add-noscript-notice-to-index.html-template branch from 646030a to 2d42b94 Compare September 10, 2020 06:41
@Blfrg
Copy link
Contributor

Blfrg commented Sep 20, 2020

The most recent recommendation has been implemented (current variable used).
@IlCallo This useful change looks like it could be merged safely.

@IlCallo
Copy link
Member

IlCallo commented Sep 21, 2020

@rstoenescu told me there could be problems and left this out on purpose. We'll wait until he can check this and write why it's better to leave it out

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

Successfully merging this pull request may close these issues.

None yet

5 participants