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

Supprime deux dépendances non utilisées #6163

Merged
merged 1 commit into from
Sep 19, 2021

Conversation

Situphen
Copy link
Member

Supprime deux dépendances non utilisées depuis le passage de Google Analytics à Matomo

QA :

  • source zdsenv/bin/activate && make update && make zmd-start && make run-back
  • Vérifier que le site web fonctionne correctement

@coveralls
Copy link

coveralls commented Sep 12, 2021

Coverage Status

Coverage remained the same at 86.621% when pulling f6d6e6c on Situphen:old-python-deps into fe854d9 on zestedesavoir:dev.

@philippemilink
Copy link
Member

Tu ne veux pas en profiter pour aussi faire le ménage ici ?

C'est sans lien avec la connexion avec un compte Google ?

@Situphen
Copy link
Member Author

Situphen commented Sep 14, 2021

Tu m'as fait douté donc j'ai pris le temps de vérifier :

  • google-api-python-client et oauth2client
    • Utilité : faire des requêtes vers les API de Google Analytics
    • Configuration : ZDS_APP["stats_ga_viewid"] et api_analytics_secrets.json => je vais les enlever
  • django-oauth-toolkit
    • Utilité : se connecter à notre API avec OAuth
    • Configuration : une ligne dans le dictionnaire REST_FRAMEWORK
  • social-auth-app-django et python-social-auth => python-social-auth est déprécié et sera enlevé avec Améliore la connexion aux réseaux sociaux #6003
    • Utilité : se connecter grâce aux réseaux sociaux
    • Configuration : principalement les variables SOCIAL_AUTH_*

@Arnaud-D
Copy link
Contributor

QA OK ✔️

Pour la QA, j'ai pris soin de bien désinstaller les deux paquets (en pratique je suis reparti d'un venv neuf). Je ne crois pas que l'upgrade désinstalle les choses qui ne sont pas dans le requirements.txt.

@Arnaud-D Arnaud-D merged commit 8cdd0a4 into zestedesavoir:dev Sep 19, 2021
@Situphen Situphen deleted the old-python-deps branch September 19, 2021 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants