[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-09-11 Thread harshach
Github user harshach closed the pull request at: https://github.com/apache/storm/pull/670 --- 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 is

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-09-11 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-139618704 +1 I will merge this into a new empty "asf-site" branch and work with INFRA to setup git-based publishing, which will eliminate the extra svn step to publish

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-09-11 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-139621217 @harshach This has been merged into the "asf-site" branch. Can you close this pull request? --- If your project is set up for it, you can reply to this email and have

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-09-01 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-136895532 @ptgoetz upmerged. Can you take a look. --- 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

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-17 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-131827381 @ptgoetz removed references to google fb and news widget is on top. Can you take a look. --- If your project is set up for it, you can reply to this email and have

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-12 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-130444965 I have a few things that I think need to be fixed before we publish this. But rather than enumerate every last little thing and ask you to fix them, I'd rather jump in

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-12 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-130447771 @ptgoetz if these issues are layout please list them out we can send a new patch. I don't think website layout needs to be blocked on docs or other fixes. I want to see

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-12 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-130488992 @ptgoetz will look into the fb goog calls and also generating the news feed. As far as separate branch goes thats fine by me but another issue could be that the doc

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-12 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-130452561 @harshach I'm not suggesting we create a feature branch, I'm suggesting we change the way we publish the site to make it easier for people to contribute to. I forgot to

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-11 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-130134736 @revans2 @ptgoetz waiting for you to take a look at it before we can merge it in. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-10 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-129523372 @HeartSaVioR Thanks. Will file a follow-up jira for the issues you raised. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-10 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-129636717 @ptgoetz can you take a look at it as well. --- 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

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-10 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-129340919 @harshach It seems fixed. Thanks. I'll look into it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-10 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-129460417 @harshach Looks beautiful to me. And spots I reported seems be fixed. I looked at whole things and found new points to fix. downloads.html —

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-10 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-129465676 @HeartSaVioR Thanks for the review. All links which point to javadoc are broken. Can I expect it will work when site resources are deployed to official site?

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-10 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-129472040 @harshach Agreed. Let's focus to redesign :) --- 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

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-09 Thread harshach
GitHub user harshach opened a pull request: https://github.com/apache/storm/pull/670 STORM-950. Apache Storm website redesign. You can merge this pull request into a Git repository by running: $ git pull https://github.com/harshach/incubator-storm STORM-950-V7 Alternatively

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-09 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-129286293 @revans2 how are you running this I didn't see any issues with resources or js missing 404s. @HeartSaVioR can you please check this PR . I ran though the links

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-09 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/670#issuecomment-129286465 @HeartSaVioR @revans2 my bad looks like I missed a commit last time. Please try again. Thanks --- If your project is set up for it, you can reply to this email and

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-08-05 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/663#issuecomment-128151202 @harshach I am getting the same thing as @HeartSaVioR Just loading the main page has over 20 404 errors returned, many of them are images, but also .css and .js files.

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-31 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/663#issuecomment-126848034 @harshach Many resources and document links are broken. Maybe you forgot to add files to commit? --- If your project is set up for it, you can reply to this

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-31 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/663#issuecomment-126795885 @HeartSaVioR @ptgoetz please take a look. --- 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

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-31 Thread harshach
GitHub user harshach opened a pull request: https://github.com/apache/storm/pull/663 STORM-950. Apache Storm website redesign. To run 1. Install Jekyll 2. jekyll serve -w will start a webserver at localhost:4000 You can merge this pull request into a Git repository by

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-30 Thread harshach
Github user harshach closed the pull request at: https://github.com/apache/storm/pull/654 --- 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 is

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-30 Thread harshach
GitHub user harshach opened a pull request: https://github.com/apache/storm/pull/659 STORM-950. Apache Storm website redesign. You can merge this pull request into a Git repository by running: $ git pull https://github.com/harshach/incubator-storm STORM-950-V4 Alternatively

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-30 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/659#issuecomment-126424522 @HeartSaVioR your comments from previous one addressed in this one https://github.com/apache/storm/pull/654#issuecomment-125104496 . Please take a look --- If your

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-30 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/659#issuecomment-126424575 @ptgoetz please take a look as well. --- 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

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-30 Thread harshach
Github user harshach closed the pull request at: https://github.com/apache/storm/pull/659 --- 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 is

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-27 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/654#issuecomment-125104496 I merged into my master and followed your description to run webserver. Some broken things are seen, here's a list. index.html Latest News :

[GitHub] storm pull request: STORM-950. Apache Storm website redesign.

2015-07-26 Thread harshach
GitHub user harshach opened a pull request: https://github.com/apache/storm/pull/654 STORM-950. Apache Storm website redesign. To test this. Install jekyll on OS X sudo gem install jekyll cd storm/docs/ jekyll serve -w You can merge this pull request into a Git