[DONE] Yaal Coop/Add PWA functionality to app and notifications capabilities #937

LoanR · 2023-09-05T13:44:37Z

PWA

L'application est installable sur différents os : iOS, macOS, Windows, Android, Linux...
Cette fonctionnalité se base sur la lib django-pwa qui permet de configurer la création du manifest.json et du service worker.

Notifications

Le système de base de notifications est implémenté et s'appuie sur la lib django-webpush.

Standard web push

L'usage standard est le suivant :

Accepter de recevoir des notifications doit être fait par une action de l'utilisateur. La demande sera toujours double, le navigateur doit lui aussi demander cette autorisation
Suivant le navigateur utilisé (PWA ou non), le navigateur contacte le serveur distant qui lui correspond (Mozilla Push Service avec autopush-rs pour Firefox par exemple)
Le navigateur fournit la clé VAPID publique
Le push service répond avec un endpoint sur lequel poster les notifications
Lorsque l'application veut envoyer une notification, elle envoie le contenu sur ce endpoint avec la clé VAPID privée
Le push service authentifie l'information puis envoie le contenu de la notification sur les appareils enregistrés. Le message peut avoir une durée de vie, ce qui fait que les appareils éteints pourront tout de même recevoir la notification par la suite si nécessaire.

Conditions de notification

Notification métier : fin d'encodage

Pour le moment, Esup-Pod utilise ces notifications pour un cas métier particulier : la fin de l'encodage d'une vidéo. Le flow suivi est le suivant :

Lorsqu'un utilisateur upload une vidéo, l'app demande la permission d'envoyer des notifications
Si l'utilisateur accepte, le navigateur s'inscrit au push service correspondant et le serveur va enregistrer la valeur de l'attribut accepts_notifications de l'Owner
Dans le même temps le navigateur demande l'autorisation de notifier sur l'appareil
Si l'utilisateur accepte, l'autorisation de notifier depuis cette url est enregistrée dans les paramètres du navigateur
Si ces conditions sont remplies, lorsque l'encoding d'une vidéo appartenant à l'utilisateur est terminé, une notification sera envoyé sur ses appareils enregistrés.

Conditions de demande de notification

Un utilisateur peut paramétrer sa préférence de notification depuis son volet menu utilisateur.
Si la préférence de notification d'un utilisateur n'a pas encore été définie, la demande de notification se fera pendant l'encodage d'une vidéo, sur la page d'édition de la vidéo.

Règles de notifications

Pour être notifié de la fin d'un encoding, un utilisateur doit avoir accepté les notifications (accepts_notifications à True), et au moins un de ses appareils doit être enregistré à un push service.
Il doit également donner l'autorisation a un site de notifier son appareil depuis les paramètres du navigateur. Le fait qu'un utilisateur décide finalement de retirer la permission de notification d'un site dans son navigateur ne peut pas être contrôlé, cela ne rentre pas en compte dans le choix de l'envoi d'une notification.
Si un utilisateur refuse d'être notifié (accepts_notification à False) ou qu'aucun de ses appareil n'est enregistré à un push service, il recevra un mail à la fin de l'encodage (si le paramètre EMAIL_ON_ENCODING_COMPLETION est à True)

Cas particulier

Sur iOS, pour utiliser le service de notification et les recevoir, il faut impérativement installer la PWA. C'est une limitation imposée par iOS.

Tests

Pour tester les notifications, une url est à disposition pour les superusers, /pwa
Un bouton permet d'envoyer une notification sur tous les appareils enregistrés (et autorisés à notifier) de l'utilisateur connecté.

resolves #912
cc @azmeuk

LoanR · 2023-09-05T13:59:45Z

Il y a un peu d'infos dans dockerfile-dev-with-volumes/README.adoc et dans pod/main/configuration.json
Dans ce dernier fichier, pod_version_init n'est pas précisé, ne sachant pas à quelle version la PWA sera déployée.

Y a-t-il un endroit indiqué pour documenter la PWA et les notification (ce qui a été décrit dans la PR) ?
La meilleure place semble être https://www.esup-portail.org/wiki/display/ES/esup-pod mais nous n'y avons pas accès...

ptitloup · 2023-09-06T07:45:19Z

Bonjour et merci pour votre contribution. Je viens de solliciter l'équipe en charge du développement pour faire les review. Pour la documentation, en effet, je vais ouvrir une page sur le wiki mais je ne pourrais malheureusement pas vous donner accès. Je vous propose donc de faire la doc au format markdown et je la mettrai en ligne sur le wiki. Bonne journée

Badatos

Merci pour cette fonctionnalité !
J’ai fait quelques remarques, principalement lié au QoC et à l'accessibilité.

pod/authentication/forms.py

Badatos · 2023-09-06T07:49:08Z

pod/authentication/models.py

+        verbose_name=_("Accept notifications"),
+        default=None,
+        null=True,
+        help_text=_("Get push notifications from your devices."),


il me semble qu'il serait plus juste de dire "on your devices" plutôt que "from" ?

Badatos · 2023-09-06T07:51:27Z

pod/main/settings.py

+WEBPUSH_SETTINGS = {
+    "VAPID_PUBLIC_KEY": "",
+    "VAPID_PRIVATE_KEY": "",
+    "VAPID_ADMIN_EMAIL": "[email protected]"


[email protected] risque d'être spammé si on le laisse par défaut.

Badatos · 2023-09-06T07:52:11Z

pod/main/templates/base.html

@@ -270,4 +274,3 @@ <h1 class="page_title">{{page_title|capfirst}}</h1>
 </body>

 </html>


Merci de laisser une ligne vide en fin de fichiers

Badatos · 2023-09-06T07:52:53Z

pod/main/templates/navbar.html

@@ -243,6 +243,11 @@ <h5 id="pod-navbar__menuuserLabel">{% if user.get_full_name != '' %}{{ user.get_
                  <a class="nav-item nav-link" href="{% url 'bbb:live_list_meeting' %}"><i class="bi bi-lightning-charge pod-nav-link-icon mx-1" aria-hidden="true"></i>{% trans 'Perform a BigBlueButton live' %}</a>
                {% endif %}
              {% endif %}
+              <div class="nav-item">
+                <button class="nav-link" onclick="new bootstrap.Toast(document.querySelector('#notification-toast')).show()">
+                  <i class="bi bi-bell pod-nav-link-icon mx-1"></i>{% trans 'Notifications settings' %}


ajouter aria-hidden="true" sur chaque icône bootstrap

pod/progressive_web_app/utils.py

pod/progressive_web_app/views.py

Badatos · 2023-09-06T07:59:19Z

pod/settings.py

+# PWA
+
+PWA_APP_NAME = 'Podeduc'
+PWA_APP_DESCRIPTION = (


Est-il possible de mettre la description en anglais, et d'inclure le systeme de traduction ?

(sinon, ajouter un commentaire pour le préciser)

pod/video_encode_transcript/utils.py

LoicBonavent

Bonjour,

Voici le retour après mes premiers tests.
Pour informations, j'ai mis à jour Pod, sur un environnement de test fonctionnel, via les commandes habituelles :

python3 -m pip install -r requirements.txt
make updatedb
make migrate
make statics
make compilelang
Puis, j'ai réalisé la configuration adéquate dans custom/settings_local.py en utilisant des clés générées via https://web-push-codelab.glitch.me/
WEBPUSH_SETTINGS = {
"VAPID_PUBLIC_KEY": "xxx",
"VAPID_PRIVATE_KEY": "xxxx",
"VAPID_ADMIN_EMAIL": "xxx"
}
Et final un restart du service uwsgi-pod.

Côté installation, aucun problème !

Par la suite, j'ai réalisé des tests sur différents navigateurs Firefox / Chrome / Edge sur mon PC (je testerai lundi sur un téléphone/tablette).
Voici alors mes remarques :

Quand je vais dans le menu, dans Paramètres de notification et que je clique sur Autoriser :

sur Chrome et Edge, le navigateur me demande de confirmer l'autorisation, mais cela me provoque une erreur 403 dans la Console du navigateur lors d'une requête sur https://xxx.umontpellier.fr/accounts/set-notifications/ 403 (forbidden)
sur Firefox, le navigateur ne me demande rien et cela provoque une erreur dans la Console : Uncaught (in promise) TypeError: reg is undefined
subscribe https://xxx.umontpellier.fr/static/webpush/webpush.js:95
setPushPreference https://xxx.umontpellier.fr/static/js/notification-toast.js?ver=3.3.1.2:25
onclick https://xxx.umontpellier.fr/:1

Pour continuer mes tests sur les notifications, j'ai alors modifier directement la valeur du champ authentication_owner.accepts_notifications pour mon compte (j'ai mis 1 pour les accepter).

Après cette manip, les notifications de fin d'encodage s'affichaient bien sur tous les navigateurs.

Sur Chrome/Edge, sur toutes les pages, j'ai une erreur non bloquante dans la console : Uncaught (in promise) TypeError: Failed to execute 'addAll' on 'Cache': Request failed
Sur la route /pwa, les boutons n'ont pas de style et le bouton Send notification n'est pas traduit. J'imagine que c'est seulement une page de test, c'est donc peu important.
Dans l'Administration, il manque quelques traductions (push information, User, subscription, Group).
Sur Chrome/Edge, j'ai pu installer l'application sur le bureau (sous Firefox, je n'ai pas trouvé mais il me semble que c'est lié à Firefox, qui ne l'autorise plus depuis 2 ans).
Par contre, l'application se nomme Podeduc et les icônes correspondent aux icônes "de base" (non surchargés) de Pod.
Serait-il possible de pouvoir réaliser une surcharge du nom de l'application et du(es) icône(s) ? Typiquement, via le fichier de configuration custom/setting_local.py.
Par exemple, chez nous, nous mettrions Plateforme vidéo UM comme nom d'application et l'icône correspondant à la charte de l'université.

Encore merci pour ce travail de qualité.

Bonne journée,

ptitloup · 2023-09-12T16:07:47Z

Bonjour @LoanR est-ce possible de répondre aux remarques svp ?

ptitloup · 2023-09-12T16:08:06Z

Est-ce possible d'apporter les modifications demandées ? merci bcp

LoanR · 2023-09-13T12:49:41Z

Hello @ptitloup, c'est bien pris en compte de notre côté, on est sur les modifs demandées et on enquête sur les erreurs rencontrées.
On reviendra ici dès qu'on aura suffisamment avancé ou si on a des questions.

azmeuk · 2023-09-13T16:12:50Z

Bonjour,

Je viens de pousser les modifications de codes demandées par @Badatos.

@LoicBonavent

Par la suite, j'ai réalisé des tests sur différents navigateurs Firefox / Chrome / Edge sur mon PC (je testerai lundi sur un téléphone/tablette).

Nous n'avons pas testé sur Edge car le cahier des charges listait Chrome, Firefox et Safari.

Me confirmes-tu que tu as fais tes tests depuis un Windows ?

Me confirmes-tu que tu tests depuis une URL publique en https ?

Peux-tu m'indiquer les versions des navigateurs avec lesquelles tu effectues tes tests ?

sur Chrome et Edge, le navigateur me demande de confirmer l'autorisation, mais cela me provoque une erreur 403 dans la Console du navigateur lors d'une requête sur https://xxx.umontpellier.fr/accounts/set-notifications/ 403 (forbidden)

À première vue je dirais que ça pourrait être un souci de CSRF. Vois-tu passer le token CSRF dans la requête ?

Sur Chrome/Edge, sur toutes les pages, j'ai une erreur non bloquante dans la console : Uncaught (in promise) TypeError: Failed to execute 'addAll' on 'Cache': Request failed

Le serviceworker essayait de mettre en cache un fichier avec un mauvais chemin. C'est corrigé.

Sur la route /pwa, les boutons n'ont pas de style et le bouton Send notification n'est pas traduit. J'imagine que c'est seulement une page de test, c'est donc peu important.

Effectivement nous avons fais cette simple page pour pouvoir tester la réception des notifications sans avoir besoin de mettre en ligne une vidéo. Cette page n'est disponible qu'aux super utilisateurs. Si c'est important on peut le traduire.

Dans l'Administration, il manque quelques traductions (push information, User, subscription, Group).

En effet ces éléments viennent de l'extension django-webpush. Il n'y a pas de traduction pour le moment mais une PR a été soumise à ce sujet.

Sur Chrome/Edge, j'ai pu installer l'application sur le bureau (sous Firefox, je n'ai pas trouvé mais il me semble que c'est lié à Firefox, qui ne l'autorise plus depuis 2 ans).

Il existe une extension qui permet l'installation avec Firefox : https://github.com/filips123/PWAsForFirefoxhttps://github.com/filips123/PWAsForFirefox

Par contre, l'application se nomme Podeduc et les icônes correspondent aux icônes "de base" (non surchargés) de Pod.
Serait-il possible de pouvoir réaliser une surcharge du nom de l'application et du(es) icône(s) ? Typiquement, via le fichier de configuration custom/setting_local.py.
Par exemple, chez nous, nous mettrions Plateforme vidéo UM comme nom d'application et l'icône correspondant à la charte de l'université.

On peut surcharger n'importe quel élément de configuration décrit dans django-pwa dans setting_local.py et notamment PWA_APP_NAME qui contrôle le nom d'affichage, ou bien PWA_APP_ICONS et PWA_APP_ICONS_APPLE pour qui contrôlent les icônes.

LoicBonavent · 2023-09-14T15:26:38Z

Bonjour,

Merci pour ce retour.
En effet, j'utilise bien Windows 10 et Firefox 117.0.1, Chrome 116.0 sur un serveur de test (il y a peut-être des contraintes à ce niveau) en HTTPS.
Concernant le problème 403, c'est bien le token CSRF qui ne passe pas dans la requête (par contre, j'ai bien un cookie csrftoken rempli); si je suis le seul à avoir ce problème, c'est que cela vient sûrement de mon environnement de test (je vais faire des tests supplémentaires pour vérifier ce point).
Et merci aussi pour les autres points, en particulier la configuration de django-pwa.

LoanR · 2023-09-15T10:48:39Z

Nous allons mettre ces questions de configuration de django-pwa dans la doc, ça simplifiera l'usage.

LoicBonavent · 2023-09-15T13:17:58Z

Bonjour,

Dans mon cas, je vous confirme que le problème de 403 venait de mon environnement de test et non de votre développement (la valeur du cookie csrftoken n'était pas fournie).

Encore une petite remarque, il semblerait qu'il y ait 2 traductions en double, ce qui pose problème pour la compilation :
"Don't forget to allow notifications from this website in your browser's settings!"
et
"Error during unsubscription..."

Merci.

azmeuk · 2023-09-15T13:22:33Z

Dans mon cas, je vous confirme que le problème de 403 venait de mon environnement de test et non de votre développement (la valeur du cookie csrftoken n'était pas fournie).

Merci pour l'info. Par curiosité, qu'est-ce qui a provoqué le souci ?

Encore une petite remarque, il semblerait qu'il y ait 2 traductions en double, ce qui pose problème pour la compilation.

On s'en occupe rapidement.

LoicBonavent · 2023-09-15T13:30:56Z

Je pense que cela doit être lié à la conf Nginx vis-à-vis de proxy_set_header Cookie ou quelque chose dans ce style.
Pour tester, j'ai modifié le fichier notification-toast.js comme ceci :
_var csrftoken = Cookies.get('csrftoken');
// Check if token is undefined or null
if (csrftoken == null) {
csrftoken = jQuery("[name=csrfmiddlewaretoken]").val();
}
formData.append("csrfmiddlewaretoken", csrftoken);

const response = await fetch(post_url, {
method: "POST",
body: formData,
headers: {
"X-Requested-With": "XMLHttpRequest",
"X-CSRFToken": csrftoken,
},
});_

Je pense qu'il y a également un petit problème de traduction pour le message d'encodage; j'obtiens du Franglais :) :
The vidéo "" has been encodé aux formats Web, and is now available on Pod.

AymericJak

Bonjour, quelques petits changements attendus de mon côté.
Merci

pod/locale/fr/LC_MESSAGES/django.po

pod/locale/fr/LC_MESSAGES/djangojs.po

pod/progressive_web_app/urls.py

pod/urls.py

azmeuk · 2023-09-19T09:59:19Z

Je pense qu'il y a également un petit problème de traduction pour le message d'encodage; j'obtiens du Franglais :) :
The vidéo "" has been encodé aux formats Web, and is now available on Pod.

C'est réglé.

@AymericJak

Les différents points sont réglés aussi.

LoicBonavent

D'après mes tests, cela semble concluant.
Un grand merci pour ce travail !

ptitloup

ok pour moi en review, je teste le fonctionnement

ptitloup · 2023-09-19T14:07:15Z

Ne pas prendre en compte les tests ci-dessous, j'avais oublié de deployer les fichiers statics :'(, toutes mes excuses

Test effectués sur MacOS (Mac Book Pro) : Ok sur Firefox et Safari. (pas d'installation possible)

Voici quelques retours de mes tests sur mon telephone pro sur Android.
Navigateur utilisé : Firefox
Pour l'installation, il faut aller dans le menu du navigateur puis cliquer sur installer. Il me rajoute bien un reccourci sur la page d'accueil mais l'icone est blanc :'(
j'aurai bien aimé un systeme identique aux notifications pour l'installation.
Au niveau des notifications, je clique sur Autoriser mais rien ne se passe :'(

Second test effectué sur Chrome. Il ne propose pas d'installer mais d'ajouter à l'ecran d'accueil. L'icone est bien présent (mais il me semble qu'il s'agit du favicon et non des icons fourni dans l'application)
Concernant les notifications, je clique sur paramètres de notification puis sur Autoriser mais rien ne se passe...

Avez-vous tester de votre côté ? Ai-je oublié quelque chose ? merci pour votre aide

ptitloup · 2023-09-19T14:23:25Z

En effet, cela fonctionne bcp mieux quand on déploie les fichiers statics.
Je confirme avoir réussi à installer et à recevoir les notifications sur mon environnement de test. Par contre, il faut cliquer une première fois sur autoriser, puis autoriser du côté du navigateur et cliquer à nouveau sur autoriser. Je trouve que ce n'est pas très ergonomique. Est-ce possible de voir pour récupérer l'évènement d'autorisation du navigateur et d'enregistrer directement ?
Petite question, comment est choisi le nom de l'application car de mon coté, elle s'appelle podeduc ... j'aurai préféré pod ulille !
Merci pour votre aide et bravo pour ce dev

ptitloup · 2023-09-19T14:25:47Z

enfin, est-ce possible de rajouter une entrée du menu pour installer l'application ? (dans le cas ou l'installation est possible !)

AymericJak

C'est bon pour moi.

Merci à vous pour cette PR !

ptitloup · 2023-09-20T09:04:26Z

Bonjour @LoanR , pourriez-vous me faire un retour sur les deux point remontés suite à mes tests fonctionnels :

est-ce possible de rajouter une entrée du menu pour installer l'application ? (dans le cas ou l'installation est possible !)
Est-ce possible de voir pour récupérer l'évènement d'autorisation du navigateur et d'enregistrer directement plutot que de devoir recliquer sur autoriser ?
Merci bcp

azmeuk · 2023-09-25T14:23:03Z

Bonjour @ptitloup
Désolé pour les délais de réponse. Je m'occupe de tous les points restants dans la journée de demain.
Éloi

Owners can set their notification preference. Notifications are sent when video encoding is complete. Co-authored-by: Éloi Rivard <[email protected]> Co-authored-by: Loan Robert <[email protected]>

azmeuk · 2023-09-26T13:21:41Z

@ptitloup

S'il ne l'a pas deja fait, on peut faire quelque comme cela :

serviceWorkerRegistration.pushManager.subscribe(options).then permet en effet de déclencher un traitement après acceptation des notifications par le navigateur de l'utilisateur, mais c'est une action qui est déjà exécutée par django-webpush. En cas d'accord django-webpush appelle la méthode postSubscribeObj qui enregistre en base de données les informations du device.

Si on appelle serviceWorkerRegistration.pushManager.subscribe nous-même alors il faut aussi faire un appel à postSubscribeObj, et c'était en partie ce dont je parlai avec ma question sur la réplication du code.

Par ailleurs, django-webpush prépare les options à passer à serviceWorkerRegistration.pushManager.subscribe dans sa méthode subscribe et notamment la transmission de la clé publique. C'est encore une partie du code qu'il sera nécessaire de dupliquer si l'on veut appeler serviceWorkerRegistration.pushManager.subscribe nous-même.

On peut envisager une manière non intrusive en écoutant l'évènement de souscription grâce à l'évènement pushsubscriptionchange, mais le support navigateur est encore anecdotique (et encore que je n'arrive pas à le faire fonctionner correctement avec Firefox, et que je ne suis pas sûr qu'il s'agisse de l'événement d'acceptation de la souscription ou bien juste d'une modification sur une souscription).

Je vois donc 2 options :

Implémentation de la fonctionnalité avec duplication d'une partie du code de django-webpush.
Garder le comportement actuel et ouvrir un ticket de demande de fonctionnalité sur le dépôt de django-webpush.

Tu me confirmes que j'implémente la 1 avec duplication d'une partie du code ?

azmeuk · 2023-09-26T14:32:23Z

Je viens de pousser l'affichage d'une entrée de menu permettant l'installation de la PWA pour les navigateurs qui la supportent, à savoir pour le moment Chrome.

azmeuk · 2023-09-26T14:45:00Z

Voici le texte de documentation qui peut être ajouté au wiki:

# PWA

L'application implémente quelques fonctionnalités des [applications web progressives](https://developer.mozilla.org/fr/docs/Web/Progressive_web_apps),
comme l'installation ou les notifications.

## Configuration

Les métadonnées de l'application sont pré-remplies, mais toutes les options (icônes, nom de l'application, etc.) peuvent être surchargées.
La liste exhaustive des paramètres de configuration est disponible [sur le dépôt de django-pwa](https://github.com/silviolleite/django-pwa?tab=readme-ov-file#configuration).
Notamment, le nom et la description de l'application sont contrôlés par les variables `PWA_APP_NAME` et `PWA_APP_DESCRIPTION`.

## Notifications push

Afin de permettre à l'application d'envoyer aux utilisateurs des notifications natives, il est nécessaire de générer une paire de clés VAPID, par exemple avec des outils tels que https://web-push-codelab.glitch.me/

Ensuite, les clés doivent être indiqués dans la configuration via les paramètres suivants:

```python
WEBPUSH_SETTINGS = {
    "VAPID_PUBLIC_KEY": "<clé-publique>",
    "VAPID_PRIVATE_KEY": "<clé-privée>",
    "VAPID_ADMIN_EMAIL": "[email protected]"
}

ptitloup · 2023-09-26T15:50:23Z

Bonjour Eloi,
merci pour l'implementation et ton retour.
Ok pour la doc, je vias rajouter une page dans le wiki.
Pour ta question, en fait, je cherche quelque chose d'ergonomique pour nos utilisateurs. Le fait d'avoir à cliquer sur "notification" puis de devoir autoriser puis après de rappuyer ne l'est pas, bcp d'utilisateurs ne vont pas le faire. J'opte donc pour la V1 en attendant mieux. J'aurai aimé que ce ne soit pas le cas. Merci

ptitloup · 2023-09-26T15:51:06Z

CC @azmeuk !

ptitloup · 2023-09-26T15:59:29Z

La doc est en ligne : https://www.esup-portail.org/wiki/display/ES/Mode+PWA+et+Notifications

azmeuk · 2023-09-26T16:03:05Z

Bonjour Eloi, merci pour l'implementation et ton retour. Ok pour la doc, je vias rajouter une page dans le wiki. Pour ta question, en fait, je cherche quelque chose d'ergonomique pour nos utilisateurs. Le fait d'avoir à cliquer sur "notification" puis de devoir autoriser puis après de rappuyer ne l'est pas, bcp d'utilisateurs ne vont pas le faire. J'opte donc pour la V1 en attendant mieux. J'aurai aimé que ce ne soit pas le cas. Merci

C'est bien compris. J'implémente ça demain matin.

azmeuk · 2023-09-26T16:04:19Z

La doc est en ligne : https://www.esup-portail.org/wiki/display/ES/Mode+PWA+et+Notifications

On dirait que le la mise en forme markdown n'a pas fonctionné.

ptitloup · 2023-09-26T16:05:10Z

La doc est en ligne : https://www.esup-portail.org/wiki/display/ES/Mode+PWA+et+Notifications

On dirait que le la mise en forme markdown n'a pas fonctionné.

Pas grave, je referais cela tranquillement

ptitloup · 2023-09-26T16:09:22Z

Voilà, c'est fait

ptitloup · 2023-09-27T10:45:16Z

Bonjour, je confirme la bonne installation de Pod. Testé via chromium sous Mac OS. Il m'a proposé de l'installer et j'ai maintenant l'application de disponible directement. Etrange, l'application se nomme podeduc !

azmeuk · 2023-09-27T12:11:49Z

Etrange, l'application se nomme podeduc !

@ptitloup as-tu configuré la variable PWA_APP_NAME ? Vois-tu les modifications dans https://ton-installation.tld/manifest.json ?

ptitloup · 2023-09-27T12:15:49Z

Quand je modifie la variable PWA_APP_NAME, je vois bien la modification. Donc tout est ok mais pourquoi PodEduc par défaut ? c'est plutot ca ma question :)

azmeuk · 2023-09-27T12:16:54Z

Je ne sais plus d'où ce nom vient. Tu préfères un autre nom par défaut ?

ptitloup · 2023-09-27T12:18:29Z

Pod Educ est le nom de l'instance Pod de l'education nationale. Certes, c'est une grosse instance mais nous sommes plutot enseignement supérieur. Ne voulant froisser personne, on pourrait mettre Pod tout simplement.

ptitloup · 2023-09-27T12:20:32Z

ok j'ai trouvé, tout se trouve dans pod/settings.py, est-ce possible de déplacer ces paramètres dans pod/main/settings.py et de remplacer podeduc par Pod svp ? Merci bcp

ptitloup

ok pour moi

azmeuk · 2023-09-27T14:21:59Z

J'ai poussé les modifications que tu as demandé tout à l'heure, et aussi un changement sur la partie front pour l'activation des notifications.

Activation / désactivation des notifications par l'utilisateur.

L'utilisateur active les notifications dans l'interface, mais les refuse dans le navigateur : pas le choix, il faut aller les réactiver à la main.

ptitloup · 2023-09-27T14:25:09Z

testé sur macOS, installation ok et notifications ok. Merci pour ces changements. Tout cela me convient.

ptitloup · 2023-09-27T14:27:32Z

Merci bcp !

…ilities (EsupPortail#937) * Add PWA functionality to app and notifications capabilities Owners can set their notification preference. Notifications are sent when video encoding is complete. Co-authored-by: Éloi Rivard <[email protected]> Co-authored-by: Loan Robert <[email protected]> * Implement a PWA installation menu entry --------- Co-authored-by: Éloi Rivard <[email protected]>

LoanR force-pushed the pwa-django-webpush branch from e7b4421 to e4719f0 Compare September 5, 2023 13:54

LoanR force-pushed the pwa-django-webpush branch from e4719f0 to e5d707d Compare September 5, 2023 14:02

Badatos self-requested a review September 6, 2023 07:53

Badatos assigned LoanR Sep 6, 2023

Badatos requested changes Sep 6, 2023

View reviewed changes

LoicBonavent self-requested a review September 8, 2023 12:20

LoicBonavent requested changes Sep 8, 2023

View reviewed changes

azmeuk force-pushed the pwa-django-webpush branch from e5d707d to 57c8997 Compare September 13, 2023 16:12

AymericJak suggested changes Sep 18, 2023

View reviewed changes

azmeuk force-pushed the pwa-django-webpush branch from 57c8997 to 08ace82 Compare September 19, 2023 09:57

LoicBonavent self-requested a review September 19, 2023 11:36

LoicBonavent approved these changes Sep 19, 2023

View reviewed changes

ptitloup reviewed Sep 19, 2023

View reviewed changes

AymericJak approved these changes Sep 20, 2023

View reviewed changes

Add PWA functionality to app and notifications capabilities

e0e46e0

Owners can set their notification preference. Notifications are sent when video encoding is complete. Co-authored-by: Éloi Rivard <[email protected]> Co-authored-by: Loan Robert <[email protected]>

azmeuk force-pushed the pwa-django-webpush branch from 6113409 to e0e46e0 Compare September 26, 2023 12:19

Implement a PWA installation menu entry

b6210e6

azmeuk force-pushed the pwa-django-webpush branch from cc485c5 to b6210e6 Compare September 27, 2023 14:10

ptitloup approved these changes Sep 27, 2023

View reviewed changes

ptitloup merged commit afb1f34 into EsupPortail:develop Sep 27, 2023
4 checks passed

azmeuk mentioned this pull request Sep 27, 2023

Correction de l'enregistrement de la préférence de notification #964

Merged

		@@ -270,4 +274,3 @@ <h1 class="page_title">{{page_title\|capfirst}}</h1>
		</body>

		</html>

[DONE] Yaal Coop/Add PWA functionality to app and notifications capabilities #937

[DONE] Yaal Coop/Add PWA functionality to app and notifications capabilities #937

Conversation

LoanR commented Sep 5, 2023 • edited Loading

PWA

Notifications

Standard web push

Conditions de notification

Notification métier : fin d'encodage

Conditions de demande de notification

Règles de notifications

Cas particulier

Tests

LoanR commented Sep 5, 2023

ptitloup commented Sep 6, 2023

Badatos left a comment

Choose a reason for hiding this comment

Badatos Sep 6, 2023

Choose a reason for hiding this comment

Badatos Sep 6, 2023

Choose a reason for hiding this comment

Badatos Sep 6, 2023 • edited Loading

Choose a reason for hiding this comment

Badatos Sep 6, 2023 • edited Loading

Choose a reason for hiding this comment

Badatos Sep 6, 2023

Choose a reason for hiding this comment

Badatos Sep 6, 2023

Choose a reason for hiding this comment

LoicBonavent left a comment

Choose a reason for hiding this comment

ptitloup commented Sep 12, 2023

ptitloup commented Sep 12, 2023

LoanR commented Sep 13, 2023

azmeuk commented Sep 13, 2023 • edited Loading

LoicBonavent commented Sep 14, 2023

LoanR commented Sep 15, 2023

LoicBonavent commented Sep 15, 2023

azmeuk commented Sep 15, 2023

LoicBonavent commented Sep 15, 2023

AymericJak left a comment

Choose a reason for hiding this comment

azmeuk commented Sep 19, 2023

LoicBonavent left a comment

Choose a reason for hiding this comment

ptitloup left a comment

Choose a reason for hiding this comment

ptitloup commented Sep 19, 2023 • edited Loading

ptitloup commented Sep 19, 2023

ptitloup commented Sep 19, 2023 • edited Loading

AymericJak left a comment

Choose a reason for hiding this comment

ptitloup commented Sep 20, 2023

azmeuk commented Sep 25, 2023

azmeuk commented Sep 26, 2023

azmeuk commented Sep 26, 2023

azmeuk commented Sep 26, 2023

ptitloup commented Sep 26, 2023

ptitloup commented Sep 26, 2023

ptitloup commented Sep 26, 2023

azmeuk commented Sep 26, 2023

azmeuk commented Sep 26, 2023

ptitloup commented Sep 26, 2023

ptitloup commented Sep 26, 2023

ptitloup commented Sep 27, 2023

azmeuk commented Sep 27, 2023

ptitloup commented Sep 27, 2023

azmeuk commented Sep 27, 2023

ptitloup commented Sep 27, 2023

ptitloup commented Sep 27, 2023

ptitloup left a comment

Choose a reason for hiding this comment

azmeuk commented Sep 27, 2023 • edited Loading

ptitloup commented Sep 27, 2023

ptitloup commented Sep 27, 2023

LoanR commented Sep 5, 2023 •

edited

Loading

Badatos Sep 6, 2023 •

edited

Loading

Badatos Sep 6, 2023 •

edited

Loading

azmeuk commented Sep 13, 2023 •

edited

Loading

ptitloup commented Sep 19, 2023 •

edited

Loading

ptitloup commented Sep 19, 2023 •

edited

Loading

azmeuk commented Sep 27, 2023 •

edited

Loading