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


Reply via email to