ophirofox icon indicating copy to clipboard operation
ophirofox copied to clipboard

(tentative d') Ajout BU UJM

Open dominique-r opened this issue 2 years ago • 23 comments

Bonjour,

J'ai essayé de suivre les instructions pour ajouter "un nouveau partenaire Europresse" mais je ne suis pas sur d'avoir fait correctement les choses (le déroulé n'est pas exactement le même qu'indiqué ...) Il y a bien un patch proposé sur un fork https://github.com/lovasoa/ophirofox/compare/master...dominique-r:ophirofox:patch-1

mais je ne le vois rien dans les pull-request

Par ailleurs j'aurais deux questions : 1/ sur le manifest.json que j'avais bidouillé en locale pour tester il y avait une entrée "https://nouveau.europresse.com/*", au niveau "optional_permissions": [ Elle me parait indispensable pour l'UJM compte tenu que nous ne sommes pas derrière un EZproxy, mais en accès sous controle de l'adresse IP Je l'ai donc rajouté dans ma proposition de modif. A moins que ce soit l'entrée "webNavigation", qui joue se rôle ?

2/ il est indiqué "Conservez l'ordre alphabétique des universités." sauf que j'ai eu l'impression qu'il y avait plusieurs type d'ordre alphabétique mélangés : un qui ne tient pas compte des article et un qui en tient compte. Ce qui après coup n'est pas le cas, c'est un ordre alpha qui ne tient pas compte des articles et donc j'ai mal classé la nouvelle entrée ! : -
... j'ai essayé de corriger ...

dominique-r avatar Sep 07 '23 14:09 dominique-r

Très bien, vous avez tout fait comme il faut jusqu'ici. Il ne vous manque plus qu'à cliquer sur le gros bouton vert create pull request

image

Si votre accès se fait directement sur nouveau.europresse.com, il n'y a rien à ajouter dans optional_permissions, le site principal d'europresse est déjà dans les permissions obligatoires (non optionnelles).

lovasoa avatar Sep 07 '23 14:09 lovasoa

Vous pouvez continuer à travailler dans votre branche sur github (https://github.com/dominique-r/ophirofox/tree/patch-1) après avoir ouvert la pull request, cette dernière se mettra à jour automatiquement.

lovasoa avatar Sep 07 '23 14:09 lovasoa

Bonjour,

Suite à la communication de la version de test de l'extension : Okay pour la partie installation temporaire de l'extension. Par contre au moment de la sélection de l'université dans les préférences de l'extension, je ne peux pas cocher l'option l'UJM. Toutes les autres universités, oui mais Saint Etienne non ... ?

Cordialement

dominique-r avatar Sep 08 '23 10:09 dominique-r

Que se passe-t-il lorsque vous cliquez sur l'université ?

lovasoa avatar Sep 08 '23 10:09 lovasoa

Rien ! je ne peux pas cocher le bouton radio.

dominique-r avatar Sep 08 '23 10:09 dominique-r

Un message que je n'avais pas vu au niveau de la fenêtre about:debugging#/runtime/this-firefox

Détails de l’avertissement Reading manifest: Warning processing background.type: An unexpected property was found in the WebExtension manifest.

dominique-r avatar Sep 08 '23 10:09 dominique-r

Rien !

D'accord. Je vais regarder ce qu'il se passe.

Reading manifest: Warning processing background.type: An unexpected property was found in the WebExtension manifest.

ça c'est juste parce que vous avez une ancienne version de firefox, je pense.

lovasoa avatar Sep 08 '23 11:09 lovasoa

| parce que vous avez une ancienne version de firefox, je pense. hummm ... je teste avec un FF Browser Extended Support Release 102.15.0esr (64bits) - Firefox est à jour Mais si c'est sans lien avec le pb okay.

dominique-r avatar Sep 08 '23 12:09 dominique-r

Oui, voilà. À propos de firefox 102, le pb est décrit là, il ne se produit pas avec les versions de firefox >= 105. C'est sans lien avec ce qui nous intéresse ici, je pense.

lovasoa avatar Sep 08 '23 12:09 lovasoa

J'ai fait deux tests (recréer deux ophirofox.xpi localement à partir du ophirofox.xpi officiel )

le premier en modifiant la saisie des caractères accentués: \u00e9 au lieu de é et E simple) même bugg

le second, j'ai ajouté (comme la première fois) "https://nouveau.europresse.com/*", dans le bloc "optional_permissions": [

Et là tout fonctionne ! ....

dominique-r avatar Sep 08 '23 13:09 dominique-r

Oui, je vois. Mais étant déjà dans les permissions obligatoires, j'aimerais bien que l'extension fonctionne sans avoir à le dupliquer dans les permissions optionnelles. Je vais faire une proposition de fix ajd.

lovasoa avatar Sep 08 '23 14:09 lovasoa

C'est le fait que nous soyons avec une url https://nouveau.europresse.com/* (puisque contrôle par IP) qui a mis à jour ce pb.

dominique-r avatar Sep 08 '23 14:09 dominique-r

J'ai intégré votre contribution. Est-ce que vous pourriez me confirmer que la dernière version fonctionne avec st étienne ?

lovasoa avatar Sep 12 '23 21:09 lovasoa

Bonjour, Je viens de l'installer sur un FF 115.2.1esr (64 bits) : et il y a deux problèmes : 1/ pas d'affichage des noms d'établissements mais seulement de leur urls exemple Autorisations facultatives pour des fonctionnalités supplémentaires : Accéder à l’activité du navigateur pendant la navigation Accéder à vos données pour https://nouveau-europresse-com.ezproxy.univ-catholille.fr Accéder à vos données pour https://nouveau-europresse-com.ezproxy.upf.pf etc et pas de module de recherche

2/ je n'ai pas retrouvé l'url de Saint Etienne

???

dominique-r avatar Sep 13 '23 09:09 dominique-r

Je pense que vous n'avez pas installé la dernière version. La dernière est la 2.4.25857.16101

Vous pouvez voir le numéro de version installé sur firefox sur la page about:addons

image

image

lovasoa avatar Sep 13 '23 09:09 lovasoa

je viens de retenter, toujours sur un FF 115.2.1esr (64 bits) et bien que je vienne d'installer la version 2.4.25859.12925, je n'ai pas du tout le même affichage que vous :

ophirofox_1 ophirofox_2

dominique-r avatar Sep 14 '23 14:09 dominique-r

étrange, ça ressemble à un problème dans firefox.

Si vous inspectez l'extension (about:debugging#/runtime/this-firefox), est-ce que vous voyez des erreurs dans cette console au moment où vous ouvrez les préférences de l'extension ?

image

lovasoa avatar Sep 14 '23 18:09 lovasoa

Je viens de retester sur un autre PC : autre version de windows (Windows 10 Entreprise versus) et autre version de FF (102.15.1esr (64 bits) ) et là pas de problème !!

Mercredi, où j'aurais de nouveau accès à l'autre PC, j’essaierais la manip indiqué.

dominique-r avatar Sep 18 '23 08:09 dominique-r

Test sur une config OS windows 7 professionnel service pack 1 (64 bits) navigateur FF Extended support release 115.2.1esr (64 bits) (FF à jour)

avec l'extension Release 2.4.25859.12925

Je ne voie rien au niveau de l'inspection de l'extension quand j'ouvre sa config (permission) et que je sélectionne ou désélectionne un bouton radio pb_config_Ophirofox_1 pb_config_Ophirofox_2

dominique-r avatar Sep 20 '23 08:09 dominique-r

Et l'onglet "préférences" n'apparaît pas ?

lovasoa avatar Sep 20 '23 12:09 lovasoa

non, je vois que les onglets Détails et Permissions

dominique-r avatar Sep 20 '23 12:09 dominique-r

@dominique-r, on notera qu'actuellement, la version de Firefox 115 sera la dernière version prise en charge pour les personnes utilisant Windows 7, Windows 8 ou Windows 8.1. Autrement dit, au quatrième quadrimestre de 2024, même si la prochaine version d'ESR pourrait contenir un correctif pour un potentiel bug, vous ne pourrez pas en profiter.

Je ne pense pas que des forks resteront compatibles avec Windows 7. Si la mise à jour du système d'exploitation n'est pas possible (mauvaise idée), il faudra peut-être regarder du côté de https://github.com/Skulltrail192/One-Core-API-Binaries

This repositry contains binary releases of the One-Core-API project. These are compatible with Windows Server 2003 SP2, Windows XP SP3 and Windows XP x64 SP2.

C'est très pointilleux, mais en quelques mots : j'ai vu Firefox 115 ESR tourner sur Windows XP, je ne sais pas si c'est totalement utilisable et conseillé de rester... sur XP.


@lovasoa il y a peut-être une explication ici : https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/options_ui

browser_style, deprecated in Manifest V3. Optional, defaulting to:

  • true in Manifest V2 and prior to Firefox 115 in Manifest V3.
  • false in Manifest V3 from Firefox 115.

Do not set browser_style to true: its not support in Manifest V3 from Firefox 118. See Manifest V3 migration for browser_style.

lnoss avatar Jan 26 '24 07:01 lnoss

Ai essayé avec la dernière version d'ESR, et la version 115.2.1 ESR citée, impossible de reproduire le problème d'interface.

Les occurrences que je trouve sur des espaces de discussion laissent penser que cela peut être un problème de profil. Saisir about:profiles dans la barre URL, redémarrer sans extensions puis avec pourrait éventuellement aider. Sinon, tenter de créer un autre profil ?

Documentation officielle liée : https://support.mozilla.org/fr/kb/gestionnaire-profils-creer-supprimer-changer-profils-firefox

lnoss avatar Jan 26 '24 10:01 lnoss