jackye1995 commented on pull request #2: URL: https://github.com/apache/iceberg-docs/pull/2#issuecomment-994422440
Also tried the website, thanks for continuing the work! It seems like some links are still broken as Ryan points out. I have some additional nitpicking: 1. Github and Slack buttons should open in a new page instead of loading in the current page. 2. the time travel script is setting a table property to perform time-travel, can we use the latest time travel syntax `AS OF`? 3. for the issue that there is no link for those top level tabs like "Integration", I think we can just disable the link? Because the 2 that has link (Spark and Flink) are all linking to the getting started page, I think we can make them just a new sub-tab called `Getting started` and disable all top level tab links. 4. Once I am in the doc page, I cannot go back to the landing page anymore, can we at least have a way to go back? 5. also agree that feature list is not full, we can think about the list of features together -- 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: dev-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
