[GitHub] cloudstack-www pull request: Improvements to survey.html

2015-10-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-www/pull/16 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] cloudstack-www pull request: Improvements to survey.html

2015-10-13 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack-www/pull/16#issuecomment-147640430 you can merge. but make sure to merge in the other branch as well, otherwise it won't go live. --- If your project is set up for it, you can reply to this email a

[GitHub] cloudstack-www pull request: Improvements to survey.html

2015-10-13 Thread terbolous
Github user terbolous commented on the pull request: https://github.com/apache/cloudstack-www/pull/16#issuecomment-147637056 I didn't touch the html manually, but included the generated html in the commit. Not sure if that is necessary or not. Should I wait for a couple of oth

[GitHub] cloudstack-www pull request: Improvements to survey.html

2015-10-13 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack-www/pull/16#issuecomment-147636042 +0 I don't think we need two LGTM etc for this. But it's automatically built, so you don't need to touch the .html. Jut edit the .md file, and build

[GitHub] cloudstack-www pull request: Improvements to survey.html

2015-10-12 Thread terbolous
GitHub user terbolous opened a pull request: https://github.com/apache/cloudstack-www/pull/16 Improvements to survey.html Made the survey.html agnostic to protocol used, previously the iframe was hardcoded with a http:// link and this breaks when visiting the site over https.