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

Only show update popup if update is available #139

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ronangaillard
Copy link

Actuellement le front cherche s'il y a une màj de dispo même si on est dans un navigateur (ie. pas dans une PWA). Ce qui n'a pas de sens.

Cette PR permet de ne pas faire la recherche de màj si la page est affichée dans un navigateur. Au passage elle fix la fonction

window.isUpdateAvailable.then((isAvailable) => {
  if(isAvailable) {
    $('#reload-btn').addEventListener('click', () => window.location.reload())
    $('#update-alert').classList.remove('d-none')
  }
})

qui ne prenait pas en compte isAvailable.

@ronangaillard ronangaillard changed the title No update checking in browser Disable update checking in browser Nov 6, 2020
@nyroDev
Copy link

nyroDev commented Nov 7, 2020

Pourquoi cela n'a pas de sens de vérifier une mise à jour dans le navigateur ?
Le service Worker sera installé aussi dans un affichage normal navigateur, donc il faut garder la mise à jour...

@ronangaillard
Copy link
Author

Le problème aujourd'hui est qu'au chargement de la page, cette dernière affiche une popup demandant de faire un mise à jour. Ce qui n'a pas beaucoup de sens.

Pour le service worker dans le doute j'ai retiré le check de l'environnement.

En revanche je laisse le if qui permet de vérifier qu'on a vraiment une maj avant de la pousser à l'utilisateur.

@ronangaillard ronangaillard changed the title Disable update checking in browser Only show update popup if update is available Nov 7, 2020
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

Successfully merging this pull request may close these issues.

2 participants