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

integration des fonctions weda comme les notifications et les capabilities #252

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

pazimor
Copy link
Contributor

@pazimor pazimor commented Sep 24, 2024

Intégration de l'objet window.weda dans l'extension.

Bonjour,

j'ai créé cette PR pour vous donner un maximum d'informations et vous faire gagner du temps dans l'implémentation.
Ces variables sont importantes pour récupérer les fonctionnalités exposées par Weda.

Je me permets de vous laisser de la documentation WF_Scripts/README.md (exemples de code etc...)
afin de vous aider dans votre implementation.

Attention

  • la partie window.weda.wedaHelper n'est pas encore déployée côté Weda.
  • l'extension n'est compatible qu'avec secure.weda.fr je n'ai donc pas pu tester sur mon Weda local, je vous laisse revenir vers moi si besoin

@Refhi
Copy link
Owner

Refhi commented Sep 28, 2024

Bonjour,

super, merci pour ce travail ! on en parle lors de notre réunion jeudi

je regarde plus en détail asap

pour le tester sur le local, je pense que changer l'ensemble des itérations de secure.weda.fr présentes dans le code (y compris dans manifest.json) pour le remplacer par azer.weda.fr (ou autre) devrait fonctionner ?

Cordialement,

@Refhi
Copy link
Owner

Refhi commented Sep 28, 2024

#253

@Refhi
Copy link
Owner

Refhi commented Oct 2, 2024

la modification du manifest [edit seulement certaines parties] implique la désactivation de l'extension chez les utilisateurs jusqu'à ce qu'ils accordent à nouveau leur autorisation.
Cf.
https://developer.chrome.com/docs/extensions/develop/concepts/permission-warnings?hl=fr
https://developer.chrome.com/docs/extensions/reference/permissions-list?hl=fr

Je ne crois pas avoir vu dans le code de chose nécessitant

  "web_accessible_resources": [
    {
      "resources": [
        "FW_scripts/*"
      ],
      "matches": [
        "<all_urls>"
      ]
    }
  ]

=> Peut-être peut-on s'en passer et simplement ajouter dans getOption un check de weda.capabilities pour désactiver les options déjà implémentées chez Weda ?

@Refhi Refhi deleted the branch Refhi:dev-2.7 October 2, 2024 19:18
@Refhi Refhi closed this Oct 2, 2024
@Refhi Refhi reopened this Oct 2, 2024
@Refhi Refhi merged commit c07d23f into Refhi:dev-2.7 Oct 2, 2024
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