Skip to content

Conversation

sudoloic
Copy link

This is an update of the Symfony deployment on Microsoft Azure for the new portal. Screenshots and some steps were outdated.

Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

👍

@sudoloic you did a great work here! Updating all those screenshots should have taken you a lot of time. We truly appreciate it!

I've added a commit to your pull request to make some long lines shorter. We have a soft-limit of 80 chars per line and some of the original lines were too long. Thanks!

.. code-block:: xml
<!-- web.config -->
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Member

Choose a reason for hiding this comment

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

@sudoloic Was this change intended?

Copy link
Author

Choose a reason for hiding this comment

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

Yep. IIS was crashing with line 425.

@xabbuh
Copy link
Member

xabbuh commented Nov 28, 2016

Thank you @sudoloic.

xabbuh added a commit that referenced this pull request Nov 28, 2016
…ts for the new Microsoft Azure Portal (sudoloic, javiereguiluz) This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #7145). Discussion ---------- Microsoft Azure update step by step article and screenshots for the new Microsoft Azure Portal This is an update of the Symfony deployment on Microsoft Azure for the new portal. Screenshots and some steps were outdated. Commits ------- 50c33a8 Make some long lines shorter to fit our soft limit of 80 chars per line 9724caf Microsoft Azure update step by step article and screenshots for the new Microsoft Azure Portal
@xabbuh xabbuh closed this Nov 28, 2016
@sudoloic sudoloic deleted the update_azure_deployment branch November 28, 2016 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

4 participants