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

Activer l'asset-extractor et corriger l'affichage d'image sur pix.org #250

Merged
merged 3 commits into from
Mar 5, 2021

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Mar 3, 2021

🦄 Problème

Les URL https://pix.org/_assets contenant les images extraites dans le bundle sont automatiquement réécrites via la configuration nginx en https://pix.org/fr/_assets (ajout de la langue)

🤖 Solution

Ajouter une exception sur _assets dans les réécriture d'URL de la configuration nginx

🌈 Remarques

  1. Un test a été ajouté afin de vérifier le bon fonctionnement de la conf. nginx
  2. Nous avons supprimé l'URL .org de pix-pro dans le message de déploiement des PRs.

💯 Pour tester

Aller sur https://site-pr250.review.pix.org et vérifier que les images s'affichent.

@pix-service
Copy link

I'm deploying this PR to these urls:

Please check it out!

@bpetetot bpetetot added the cross-team Toutes les équipes de dev label Mar 3, 2021
@bpetetot bpetetot changed the title Activate assets extractor Activer l'asset-extractor et corriger l'affichage d'image sur pix.org Mar 3, 2021
@pix-service-auto-merge pix-service-auto-merge merged commit 074b822 into dev Mar 5, 2021
@pix-service-auto-merge pix-service-auto-merge deleted the add-assets-extractor branch March 5, 2021 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants