bitsondatadev commented on code in PR #8919:
URL: https://github.com/apache/iceberg/pull/8919#discussion_r1389531071
##########
site/mkdocs.yml:
##########
@@ -23,33 +23,37 @@ theme:
logo: assets/images/iceberg-logo-icon.png
favicon: assets/images/favicon-96x96.png
features:
+ - content.tabs.link
+ - content.code.copy
+ - content.code.annotate
- navigation.tabs
- navigation.path
- navigation.top
- navigation.tracking
- - toc.follow
+ - offline
- search.suggest
- search.highlight
- - content.tabs.link
- - content.code.copy
- - content.code.annotate
+ - toc.follow
plugins:
- search
- macros:
include_yaml:
- variables.yml
- monorepo
+ - offline:
+ enabled: !ENV [OFFLINE, false]
nav:
- Quickstart:
- Spark: spark-quickstart.md
- Hive: hive-quickstart.md
- Docs:
- nightly: '!include docs/docs/nightly/mkdocs.yml'
-# - latest: '!include docs/docs/latest/mkdocs.yml'
-# - 1.3.1: '!include docs/docs/1.3.1/mkdocs.yml'
-# - 1.3.0: '!include docs/docs/1.3.0/mkdocs.yml'
+ - latest: '!include docs/docs/latest/mkdocs.yml'
Review Comment:
I've gotten mixed signals depending who I've spoken to about meta refresh
for redirects.
According to most of the stuff that comes up it's [not as big of a
deal](https://webmasters.stackexchange.com/a/141240) as some made it out to be.
So we can likely change `latest` to being a refresh page, but I'm curious to
see how mkdocs renders that.
We should ideally have latest get ranked by Google and we may just want to
be cognizant if we get a bunch of random versions or if latest gets ranked
higher.
It's a bit of lore and black box when it comes to ensuring a certain version
gets indexed and ranked by Google. No real way to control it so this was my
feeble attempt.
<https://developers.google.com/search/docs/crawling-indexing/301-redirects#metarefresh>
Based on their docs, I think we should be fine if we update all the pages to
set the [`site_url`](https://www.mkdocs.org/user-guide/configuration/#site_url)
which will always point to latest.
Also, we'll likely want to add an [outdated version
warning](https://squidfunk.github.io/mkdocs-material/setup/setting-up-versioning/#version-warning)
which will link to the "latest" equivalent of the page if available, thus
sending yet another signal to Google not to rank anything that's not the latest
higher.
Okay, all that said, yeah...I think we can remove that and just use a meta
refresh and it hopefully won't affect search engine rankings.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
