epugh commented on code in PR #891: URL: https://github.com/apache/solr/pull/891#discussion_r906195750
########## README.md: ########## @@ -28,7 +28,7 @@ result faceting and highlighting. ## Online Documentation This README file only contains basic setup instructions. For more -comprehensive documentation, visit <https://solr.apache.org/guide/> +comprehensive documentation, visit <https://solr.apache.org/guide/solr> Review Comment: I tested, and `https://solr.apache.org/guide/` works, `https://solr.apache.org/guide/solr` doesn't.... ########## README.md: ########## @@ -28,7 +28,7 @@ result faceting and highlighting. ## Online Documentation This README file only contains basic setup instructions. For more -comprehensive documentation, visit <https://solr.apache.org/guide/> +comprehensive documentation, visit <https://solr.apache.org/guide/solr> Review Comment: Nevermind, I see you made a change later on to make the urls have /solr/ ;-) ########## README.md: ########## @@ -28,7 +28,7 @@ result faceting and highlighting. ## Online Documentation This README file only contains basic setup instructions. For more -comprehensive documentation, visit <https://solr.apache.org/guide/> +comprehensive documentation, visit <https://solr.apache.org/guide/solr> Review Comment: However, https://solr.apache.org/guide/solr/latest/configuration-guide/solr-plugins.html works, and https://solr.apache.org/guide/latest/configuration-guide/solr-plugins.html doesn't, so maybe this is okay? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
