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

URLs to archives should be https #556

Closed
pardahlman opened this issue Apr 11, 2017 · 2 comments
Closed

URLs to archives should be https #556

pardahlman opened this issue Apr 11, 2017 · 2 comments
Milestone

Comments

@pardahlman
Copy link
Contributor

Hello team,

Beautifully structured documents 👌 , thanks a lot for making it easy to getting started with Akeneo PIM 🎉 .

I found a really trivial problem in the docs - in fact, I noticed it a few days ago and thought I'd just let it go... but I've been thinking about it every now and then, so here goes!

I'm following the guide for Installing Akeneo PIM CE with the Archive. As I was writing scripts to download and install Akeneo PIM, I noticed that the download URLs for the archives returned 301 Moved Permanently. The reason for this that the download site uses https, but the documents uses http. This halted my script, as I've disabled auto redirects.

The fix for this is just adding two characters, so very trivial!
Thanks and sorry! 🙂

@nidup
Copy link
Contributor

nidup commented Apr 11, 2017

Hi @pardahlman

Thank you for the feedback! Don't hesitate to open a pull request to change the "http" to "https" in the file installation_ce_archive.rst, on the 1.7 branch, we'll merge and deploy it 🎉

Have a good day,

pardahlman added a commit to pardahlman/pim-docs that referenced this issue Apr 11, 2017
@pardahlman
Copy link
Contributor Author

Said and done! #558 fixes this!

@nidup nidup added this to the 1.8.x milestone Apr 11, 2017
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

No branches or pull requests

2 participants