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

#3620 Fix url images import #3761

Merged
merged 4 commits into from
Aug 12, 2016

Conversation

ChantyTaguan
Copy link
Contributor

@ChantyTaguan ChantyTaguan commented Aug 2, 2016

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) #3620

QA

  • Créez un article/tuto via la fonction d'import, avec des images
  • Vérifier que les images sont bien insérées avec le lien https

Je joints un zip contenant un article formatté correctement pour l'import (avec référence archive: pour les images, et un zip avec les images référencées maos n'hésitez pas à tester avec un autre contenu (un tuto par exemple).

arbres-phylo.zip
images.zip

@coveralls
Copy link

coveralls commented Aug 2, 2016

Coverage Status

Coverage decreased (-15.9%) to 71.807% when pulling 78a0a14 on ChantyTaguan:fix-url_images_import into c50283c on zestedesavoir:dev.

@coveralls
Copy link

coveralls commented Aug 3, 2016

Coverage Status

Coverage increased (+0.08%) to 87.753% when pulling c4182da on ChantyTaguan:fix-url_images_import into c50283c on zestedesavoir:dev.

@GerardPaligot
Copy link
Member

L'effet pervers de cette PR, c'est que les images ne s'affichent plus correctement en local puisque nous n'avons pas d'HTPPS (logique).

La QA est ok mais peut-être que tu pourrais renseigner une version http dans le settings_test_local pour secure_url pour éviter d'avoir des images cassées en local ?

@coveralls
Copy link

coveralls commented Aug 10, 2016

Coverage Status

Coverage increased (+0.08%) to 87.76% when pulling 20e3e39 on ChantyTaguan:fix-url_images_import into c50283c on zestedesavoir:dev.

@GerardPaligot
Copy link
Member

Super @ChantyTaguan ! :)

@GerardPaligot GerardPaligot merged commit 5e5bd2a into zestedesavoir:dev Aug 12, 2016
'site': {
'secure_url': u"http://127.0.0.1:8000"
}
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'arrive un peu tard, mais on a un problème là.

ZDS_APP est importé depuis settings.py. Ici, on écrase complètement le truc. Du coup aucun test ne peut passer en local.

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.

None yet

4 participants